Skip to content

Conversation

@oarbusi
Copy link
Collaborator

@oarbusi oarbusi commented Mar 13, 2025

Proposed changes

Deprecates ServerlessInstance, ServerlessPrivateEndpoint and Serverless type in CloudBackupRestoreJobs

Followup of Cloudformation deprecation mongodb/mongodbatlas-cloudformation-resources#1306

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

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 added any necessary documentation (if appropriate)
  • I have run make fmt and formatted my code
  • I have tested the CDK constructor in a CFN stack. See TESTING.md
  • 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.

Further comments

@oarbusi oarbusi requested a review from a team as a code owner March 13, 2025 09:10
* @default true` for new Atlas Organizations created with the provider to prevent infrastructure drift caused by creation of new alerts.
* @schema CfnOrganizationProps#SkipDefaultAlertsSettings
*/
readonly skipDefaultAlertsSettings?: boolean;
Copy link
Member

Choose a reason for hiding this comment

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

Would confirm with @maastha, not sure if this change has been released yet

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is being picked up in the CI (see failure), that's why I included it. If it can't be included in this I will work around this

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

talked with @maastha, I'll wait to merge and release a CDK version with these changes until next week.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@oarbusi in any case I think it's best practice you separate this. First you do a PR with these changes if they are blocking and then you do a PR for deprecation.

When I open the PR and see the title I don't expect these changes to be here.

Copy link
Collaborator Author

@oarbusi oarbusi Mar 13, 2025

Choose a reason for hiding this comment

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

@marcosuma Because both changes are now in CFN repo master branch, and how this repo works, CI checks detect both changes, and separate PRs would have failing CI checks, which is not ideal IMO. That is why I prefer to not split it if possible and to just wait. I can change this PR title to accomodate both changes

@oarbusi oarbusi changed the title chore: Deprecates ServerlessInstance, ServerlessPrivateEndpoint and Serverless type in CloudBackupRestoreJobs feat: Adds SkipDefaultAlertsSettings and GenAIFeaturesEnabled to Organization and deprecates ServerlessInstance, ServerlessPrivateEndpoint and Serverless type in CloudBackupRestoreJobs and Mar 13, 2025
@oarbusi oarbusi closed this Mar 25, 2025
@oarbusi oarbusi reopened this Mar 25, 2025
@oarbusi oarbusi changed the title feat: Adds SkipDefaultAlertsSettings and GenAIFeaturesEnabled to Organization and deprecates ServerlessInstance, ServerlessPrivateEndpoint and Serverless type in CloudBackupRestoreJobs and feat: Adds SkipDefaultAlertsSettings and GenAIFeaturesEnabled to Organization, maximumThroughput to Trigger and deprecates ServerlessInstance, ServerlessPrivateEndpoint and Serverless type in CloudBackupRestoreJobs and Mar 25, 2025
@oarbusi oarbusi merged commit c284089 into main Mar 26, 2025
14 checks passed
@oarbusi oarbusi deleted the CLOUDP-277813 branch March 26, 2025 07:58
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.

6 participants