Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(rds): Add support for replicationSourceIdentifier prop #33280

Open
2 tasks
moltar opened this issue Feb 3, 2025 · 1 comment
Open
2 tasks

(rds): Add support for replicationSourceIdentifier prop #33280

moltar opened this issue Feb 3, 2025 · 1 comment
Labels
@aws-cdk/aws-rds Related to Amazon Relational Database effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p2

Comments

@moltar
Copy link
Contributor

moltar commented Feb 3, 2025

Describe the feature

Docs: https://docs.aws.amazon.com/AmazonRDS/latest/AuroraUserGuide/AuroraPostgreSQL.Migrating.RDSPostgreSQL.Replica.html

CF docs: https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-rds-dbcluster.html#aws-resource-rds-dbcluster-properties

This prop is available on the CfnDBCluster.

Use Case

Migrating RDS types, need to set this prop.

Proposed Solution

The DatabaseCluster class, should accept replicationInstanceSource prop, which accepts IDatabaseInstance:

replicationInstanceSource: IDatabaseInstance

Other Information

  • Cannot set masterUserPassword and masterUsername (CDK sets defaults) internally.
  • The value of replicationSourceIdentifier is NOT an identifier, but a full ARN.

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

CDK version used

N/A

Environment details (OS name and version, etc.)

N/A

@moltar moltar added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Feb 3, 2025
@github-actions github-actions bot added the @aws-cdk/aws-rds Related to Amazon Relational Database label Feb 3, 2025
@khushail khushail added investigating This issue is being investigated and/or work is in progress to resolve the issue. p2 and removed needs-triage This issue or PR still needs to be triaged. labels Feb 3, 2025
@khushail khushail self-assigned this Feb 3, 2025
@khushail
Copy link
Contributor

khushail commented Feb 4, 2025

Thanks @moltar for requesting this feature. Its evident from CDK docs that this prop is supported in Cloudformation -https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_rds.CfnDBCluster.html#replicationsourceidentifier, hence can be supported via CDK too.

Marking it as P2 as the workaround of using L1 Construct already exists. Contributions from the community are also welcome.

@khushail khushail added effort/small Small work item – less than a day of effort and removed investigating This issue is being investigated and/or work is in progress to resolve the issue. labels Feb 4, 2025
@khushail khushail removed their assignment Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-rds Related to Amazon Relational Database effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p2
Projects
None yet
Development

No branches or pull requests

2 participants