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

DBCluster auto generates password #1413

Merged
merged 2 commits into from
Aug 11, 2022
Merged

Conversation

mmiranda
Copy link
Contributor

@mmiranda mmiranda commented Aug 9, 2022

Description of your changes

When a secret with the master password is not given, the resource will autogenerate an initial password

Fixes #667
Replaces #671 which seems to be stuck

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Compiled locally and tested in a sandbox cluster launching a few Aurora databases

Signed-off-by: Mateus Miranda <mateusmiranda@gmail.com>
Signed-off-by: Mateus Miranda <mateusmiranda@gmail.com>
Copy link
Member

@haarchri haarchri left a comment

Choose a reason for hiding this comment

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

LGTM thanks for Implementation

@haarchri haarchri merged commit 32ee5d8 into crossplane-contrib:master Aug 11, 2022
AlessandroBenoni pushed a commit to AlessandroBenoni/provider-aws that referenced this pull request Aug 17, 2022
* RDS DBCluster autogenerates password when secret is empty

Signed-off-by: Mateus Miranda <mateusmiranda@gmail.com>
Signed-off-by: Alessandro Benoni <alessandro@bredec.com>
@guilhermef
Copy link
Contributor

Hi @haarchri, thanks for merging this PR.
We need to merge the #1434 before releasing this feature.

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.

DBCluster spec is different from RDSInstance on requirement of masterUserPasswordRef field
3 participants