Skip to content

Conversation

@oarbusi
Copy link
Collaborator

@oarbusi oarbusi commented Mar 12, 2025

Proposed changes

Deprecates ServerlessInstance, ServerlessPrivateEndpoing and Serverless type in CloudBackupRestoreJobs .
There's no mention of M2 or M5 instance sizes in cluster, so no changes are needed in the Cluster resource

Link to any related issue(s): CLOUDP-277813

Type of change:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as
    expected)
  • This change requires a documentation update
  • If changes include removal or addition of 3rd party GitHub actions, I updated our internal document. Reach out to the APIx Integration slack channel to get access to the internal document.

Required Checklist:

  • I have signed the MongoDB CLA
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that this change does not generate any credentials and that they are NOT accidentally logged anywhere.
  • I have added any necessary documentation (if appropriate)
  • I have run make fmt and formatted my code
  • For CFN Resources: I have released by changes in the private registry and proved by change
    works in Atlas

Further comments

@oarbusi oarbusi requested a review from a team as a code owner March 12, 2025 10:11
@oarbusi
Copy link
Collaborator Author

oarbusi commented Mar 12, 2025

failing test is due to the shimming logic for serverless API that now creates flex clusters instead of serverless instances.

POST: HTTP 400 Bad Request (Error code: "NOT_SERVERLESS_TENANT_CLUSTER") Detail: The specified cluster ct-private-endpoint-serverless in group <group> is not a serverless tenant cluster. Reason: Bad Request. Params: [ct-private-endpoint-serverless <group>]

@@ -1,5 +1,7 @@
# MongoDB::Atlas::ServerlessInstance

**WARNING:** This resource is deprecated and will be removed in January 2026. For more details, see [Migrate your programmatic tools from M2, M5, or Serverless Instances to Flex Clusters](https://www.mongodb.com/docs/atlas/flex-migration/).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we tell them if they try to create it will fail because a Flex cluster will be created instead?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also I think we should add a sentence that we are planning to support Flex in CloudFormation, with a (timeline yet to be announced)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Create will not fail, but it will create a flex cluster. The linked guide already mentions that the flex clusters are created instead of shared/serverless instanced. I don't think it's necessary here IMO

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The link also states that Flex support is coming: https://www.mongodb.com/docs/atlas/flex-migration/#atlas-cloudformation-migration I don't think it's necessary to mention it in the CFN docs IMO

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

my only advice is find a way to tell customers they will experience the error like the one you got for the failing tests (link to your comment)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh okay, you were referring to ServerlessPrivateEndpoint, not ServerlessInstance. I understand now, I will add your suggestion to ServerlessPrivateEndpoint docs

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done in bdea3ca

@@ -1,5 +1,7 @@
# MongoDB::Atlas::ServerlessPrivateEndpoint

**WARNING:** This resource is deprecated and will be removed in January 2026. For more details, see [Migrate your programmatic tools from M2, M5, or Serverless Instances to Flex Clusters](https://www.mongodb.com/docs/atlas/flex-migration/).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I remember this was going to be stopped in March 2025, am I right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, after reviewing again https://www.mongodb.com/docs/atlas/flex-migration/, I see that the date has been changed to May. Changing accordingly

# MongoDB::Atlas::ServerlessPrivateEndpoint

Returns, adds, edits, and removes private endpoints for serverless instances. To learn more, see the Atlas Administration API tab on the following tutorial.
**WARNING:** This resource is deprecated and will be removed in May 2025. If you try to create a ServerlessPrivateEndpoint for a newly created ServerlessInstance, it will fail because the newly created ServerlessInstance are now Flex clusters. For more details, see [Migrate your programmatic tools from M2, M5, or Serverless Instances to Flex Clusters](https://www.mongodb.com/docs/atlas/flex-migration/). Returns, adds, edits, and removes private endpoints for serverless instances. To learn more, see the Atlas Administration API tab on the following tutorial.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
**WARNING:** This resource is deprecated and will be removed in May 2025. If you try to create a ServerlessPrivateEndpoint for a newly created ServerlessInstance, it will fail because the newly created ServerlessInstance are now Flex clusters. For more details, see [Migrate your programmatic tools from M2, M5, or Serverless Instances to Flex Clusters](https://www.mongodb.com/docs/atlas/flex-migration/). Returns, adds, edits, and removes private endpoints for serverless instances. To learn more, see the Atlas Administration API tab on the following tutorial.
**WARNING:** This resource is deprecated and will be removed in May 2025. If you try to create a new ServerlessInstance, you will create a Flex cluster. As a result, you can't create ServerlessPrivateEndpoints for these new instances. For more details, see [Migrate your programmatic tools from M2, M5, or Serverless Instances to Flex Clusters](https://www.mongodb.com/docs/atlas/flex-migration/). Returns, adds, edits, and removes private endpoints for serverless instances. To learn more, see the Atlas Administration API tab on the following tutorial.

Let's try to avoid using 'fail', and let's make sure that the cause is foregrounded. Please make this change elsewhere as it applies.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Applied in all the places it applies bbd3ce3. Thanks!

Copy link
Member

@lantoli lantoli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, can you also do a follow-up PR to update CDK?

@oarbusi
Copy link
Collaborator Author

oarbusi commented Mar 13, 2025

@lantoli yes I will do the changes in CDK aswell

@oarbusi
Copy link
Collaborator Author

oarbusi commented Mar 13, 2025

Merging, test failure is expected and docs has said that LGTM once suggested change has been applied

@oarbusi oarbusi merged commit 43583d4 into master Mar 13, 2025
34 of 35 checks passed
@oarbusi oarbusi deleted the CLOUDP-277813 branch March 13, 2025 08:18
@oarbusi oarbusi changed the title chore: Deprecates ServerlessInstance, ServerlessPrivateEndpoing and Serverless type in CloudBackupRestoreJobs chore: Deprecates ServerlessInstance, ServerlessPrivateEndpoint and Serverless type in CloudBackupRestoreJobs Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants