aws-cdk icon indicating copy to clipboard operation
aws-cdk copied to clipboard

aws-rds: Updating engine version on DBInstances that are part of a cluster causes an error

Open joshlartz opened this issue 3 years ago • 2 comments

Describe the bug

When AWS::RDS::DBInstance resources are part of a AWS::RDS::DBCluster, trying to update their engine version results in an error.

Expected Behavior

The CloudFormation update should succeed with the engine version update.

Current Behavior

The specified DB Instance is a member of a cluster. Modify the DB engine version of the DB Cluster using the ModifyDbCluster API

Reproduction Steps

Using rds.DatabaseCluster sets the engine version on the cluster as well as all of the instances it creates.

Possible Solution

Do not set engine version on the instances, only the cluster.

Additional Information/Context

No response

CDK CLI Version

2.40.0

Framework Version

No response

Node.js Version

16.10

OS

MacOS

Language

Typescript

Language Version

No response

Other information

No response

joshlartz avatar Sep 21 '22 22:09 joshlartz

Willing to fix

joshlartz avatar Sep 21 '22 22:09 joshlartz

https://github.com/aws-cloudformation/cloudformation-coverage-roadmap/issues/847

joshlartz avatar Sep 22 '22 01:09 joshlartz

@joshlartz it looks like this is the same issue that was reported in https://github.com/aws/aws-cdk/issues/21758. I'll close this one, but if you are willing to submit a fix that would be awesome!

corymhall avatar Sep 26 '22 13:09 corymhall

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see. If you need more assistance, please either tag a team member or open a new issue that references this one. If you wish to keep having a conversation with other community members under this issue feel free to do so.

github-actions[bot] avatar Sep 26 '22 13:09 github-actions[bot]

@corymhall the PR to fix it was already attached to this issue. Can you give that a review so we can get it merged?

joshlartz avatar Sep 26 '22 14:09 joshlartz