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
Incompatible change: SQL database deletion policy #13206
Incompatible change: SQL database deletion policy #13206
Comments
This shouldn't be the default behaviour- I've confirmed it was unintentional with the reviewer of the change. I'll make the change to the default, setting it back to delete. This will have impact on users roughly equivalent to a breaking change, but is correcting a bug/regression so it'll be done with a minor version rather than a major one. |
…ashicorp#6936) Fixes hashicorp/terraform-provider-google#13206 Signed-off-by: Modular Magician <magic-modules@google.com>
…ashicorp#6936) Fixes hashicorp#13206 Signed-off-by: Modular Magician <magic-modules@google.com>
…6936) (#4972) Fixes hashicorp/terraform-provider-google#13206 Signed-off-by: Modular Magician <magic-modules@google.com> Signed-off-by: Modular Magician <magic-modules@google.com>
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. |
Terraform Version
Terraform v1.3.6 / registry.terraform.io/hashicorp/google v4.45.0
Affected Resource(s)
google_sql_database
Terraform Configuration Files
Debug Output
(not necessary for this issue)
Panic Output
(no panic)
Expected Behavior
The database is deleted.
Actual Behavior
The database is just abandoned (not deleted).
Steps to Reproduce
terraform apply
terraform destroy
Important Factoids
N/A
References
This is likely caused by #13107. The PR introduces a deletion policy which is set to "ABANDON" as default. This changes the default behaviour, as "ABANDON" does not really delete databases. We had to explicitly specify another deletion policy to restore the previous behaviour:
This should be clearly documented as breaking change so that users can update their Terraform resources.
The text was updated successfully, but these errors were encountered: