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

Remove masterUserPasswordSecretRef as a required attribute #1669

Merged

Conversation

loalf
Copy link
Contributor

@loalf loalf commented Feb 21, 2023

Fixes #1668

When creating an Aurora Global cluster with more than 2 regional clusters, the password attribute is mandatory for the first one, but not for the second one. Right now, it's impossible to create global cluster with more than 2 regional cluster.

If you avoid setting up the masterUserPasswordSecretRef, Crossplane CRD will complain because this field is required and won't let you create the resource.

If you set up the masterUserPasswordSecretRef, the AWS API will complain that the cluster can't be created with the following error message

create failed: cannot create DBCluster in AWS: InvalidParameterCombination: Cannot specify password for cross region replication cluster

@loalf loalf force-pushed the rds-dbcluster-required-attribute branch from 92e7130 to 52fc0e3 Compare February 21, 2023 11:22
@loalf loalf force-pushed the rds-dbcluster-required-attribute branch from 52fc0e3 to 1bf7f80 Compare February 22, 2023 14:58
apis/rds/v1alpha1/custom_types.go Show resolved Hide resolved
apis/rds/v1alpha1/custom_types.go Outdated Show resolved Hide resolved
@loalf loalf force-pushed the rds-dbcluster-required-attribute branch from 1bf7f80 to 2c659fa Compare March 27, 2023 10:23
@loalf
Copy link
Contributor Author

loalf commented Mar 29, 2023

@MisterMX thanks for guiding me in the right direction. I've updated the PR with the required changes (or so I think 😆 ).

Signed-off-by: Javier Lopez <alcaraz1983@gmail.com>
@loalf loalf force-pushed the rds-dbcluster-required-attribute branch from 2c659fa to 90c8764 Compare April 20, 2023 12:32
@loalf
Copy link
Contributor Author

loalf commented Apr 20, 2023

PR updated with a fully example on how to create a Global Cluster per requested. Pinging @MisterMX

Copy link
Collaborator

@MisterMX MisterMX left a comment

Choose a reason for hiding this comment

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

LGTM now. Thanks a lot @loalf!

@MisterMX MisterMX merged commit 0428287 into crossplane-contrib:master Apr 24, 2023
8 checks passed
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.

spec.forProvider.masterUserPasswordSecretRef shouldn't be required when creating a secondary Aurora cluster
2 participants