-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
aws_rds_cluster from snapshot doesn't set new master password #9492
Comments
Hi @valentinbud 👋 Thank you for reporting this and sorry for the unexpected behavior. It does appear the resource is not properly handling |
Hi @bflad 👋, Thank you for your prompt response. For what is worth at the moment I do it in two Have a wonderful day 😃. |
…shot restore Reference: #9492 Since we cannot read the master password from the API during `Read`, it would silently be saved with the configuration value in the Terraform state and never trigger an update. This cannot be found via the acceptance testing framework without attempting to connect to the RDS Cluster (e.g. via the mysql provider), but relatedly added a test for master_username since that was also missing during snapshot restore, which highlighted a related missing test. ``` --- FAIL: TestAccAWSRDSCluster_SnapshotIdentifier_MasterUsername (377.68s) testing.go:568: Step 0 error: Check failed: Check 4/4 error: aws_rds_cluster.test: Attribute 'master_username' expected "username1", got "foo" --- PASS: TestAccAWSRDSCluster_SnapshotIdentifier_MasterPassword (378.00s) ``` Output from acceptance testing: ``` --- PASS: TestAccAWSRDSCluster_SnapshotIdentifier_MasterPassword (364.62s) --- PASS: TestAccAWSRDSCluster_SnapshotIdentifier_MasterUsername (387.26s) ```
The fix for this has been merged and will release later today with version 2.21.0 of the Terraform AWS Provider. 👍 |
This has been released in version 2.21.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests, documentation updates, or bug reports with this functionality, please create a new GitHub issue following the template for triage. Thanks! |
Hi @bflad 👋 I have tried it today with latest release of aws provider. It works like a charm. Thank you for fixing this. Have a wonderful day 😃 |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Community Note
Terraform Version
Affected Resource(s)
Terraform Configuration Files
Debug Output
I don't think it would help in this case.
terraform apply
successfully creates the resources.Panic Output
There's no panic output.
Expected Behavior
I expect the new RDS cluster to have the
root
password set up tocbMTnFjpnVJL5DrCtk4pdqQxgyCgUQrV123
.Actual Behavior
I have tried to connect to the cluster using
root
with passwordcbMTnFjpnVJL5DrCtk4pdqQxgyCgUQrV123
and receive the following error:Steps to Reproduce
Create an Aurora RDS cluster from a snapshot of an existing Aurora cluster with a different
master_password
setup and try to connect to the cluster endpoint with the newmaster_password
.Important Factoids
Changing the master password of the
qa-cluster
using AWS CLI or AWS Console allows me to login to the cluster.References
I have searched the open issues but couldn't find one that matches my use case exactly.
It might relate to #3220 though
Thank you for looking into this 😃.
The text was updated successfully, but these errors were encountered: