-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
fix(rds): read replica instance cannot join domain #19202
Conversation
The read replica instance always uses the same engine as the source instance but some CF validations require the engine to be explicitely passed when some properties are specified. Pass the `engine` property to the CF resource if `domain` is specified. Closes aws#18786
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fix @jogold!
Two small comments/questions.
SourceDBInstanceIdentifier: Match.anyValue(), | ||
Engine: 'postgres', | ||
}); | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder... since this is a CFN validation, do we need a new small integration test for this, to confirm it deploys?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ideally yes but it's complicated because it means we need to create a domain using L1 resources from @aws-cdk/aws-directoryservice
... and I don't have experience with those resources.
(domain
is currently a string
but could have been a IDirectory
if we had L2 resources for this)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK.
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @jogold, awesome work as always 🙂.
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
The read replica instance always uses the same engine as the source instance but some CF validations require the engine to be explicitely passed when some properties are specified. Pass the `engine` property to the CF resource if `domain` is specified. Closes aws#18786 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
The read replica instance always uses the same engine as the source instance
but some CF validations require the engine to be explicitely passed when some
properties are specified.
Pass the
engine
property to the CF resource ifdomain
is specified.Closes #18786
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license