Skip to content

fix(rds): properly handle RETAIN_ON_UPDATE_OR_DELETE in subresources#38331

Open
wasim-builds wants to merge 1 commit into
aws:mainfrom
wasim-builds:fix/rds-retain-on-update-or-delete
Open

fix(rds): properly handle RETAIN_ON_UPDATE_OR_DELETE in subresources#38331
wasim-builds wants to merge 1 commit into
aws:mainfrom
wasim-builds:fix/rds-retain-on-update-or-delete

Conversation

@wasim-builds

Copy link
Copy Markdown

Fixes #37780. Updated helperRemovalPolicy and defaultDeletionProtection to properly handle RemovalPolicy.RETAIN_ON_UPDATE_OR_DELETE rather than collapsing it. Also updated serverless-cluster.ts to use helperRemovalPolicy.

Copilot AI review requested due to automatic review settings July 16, 2026 18:36

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions github-actions Bot added beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK bug This issue is a bug. effort/medium Medium work item – several days of effort p2 labels Jul 16, 2026
@aws-cdk-automation
aws-cdk-automation requested a review from a team July 16, 2026 18:37

@aws-cdk-automation aws-cdk-automation left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The pull request linter fails with the following errors:

❌ Fixes must contain a change to a test file.
❌ Fixes must contain a change to an integration test file and the resulting snapshot.

If you believe this pull request should receive an exemption, please comment and provide a justification. A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed, add Clarification Request to a comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK bug This issue is a bug. effort/medium Medium work item – several days of effort p2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

rds: RemovalPolicy.RETAIN_ON_UPDATE_OR_DELETE not propagated to child resources in RDS constructs

3 participants