Skip to content

Conversation

@OscarVanL
Copy link
Contributor

@OscarVanL OscarVanL commented May 22, 2025

Relates to #727

Adds support for retain_backups_on_delete for mysql

@OscarVanL OscarVanL requested review from a team, ayushmjain, imrannayer and q2w as code owners May 22, 2025 10:20
@OscarVanL OscarVanL changed the title Support retain_backups_on_delete for mysql feat: Support retain_backups_on_delete for mysql May 22, 2025
@imrannayer
Copy link
Collaborator

@OscarVanL thx for the PR. Can you plz update versions.tf file? Change min provider version required to 6.31 when this feature was released.
Thanks

@OscarVanL
Copy link
Contributor Author

@imrannayer Done, is that correct?

@imrannayer
Copy link
Collaborator

@OscarVanL can you plz follow Contrib guide to fix lint issues?

Thanks

@OscarVanL
Copy link
Contributor Author

@imrannayer I hope that's good now.

@imrannayer
Copy link
Collaborator

/gcbrun

1 similar comment
@imrannayer
Copy link
Collaborator

/gcbrun

@imrannayer
Copy link
Collaborator

@OscarVanL can u plz add it to safer mysql sub-module?

@OscarVanL
Copy link
Contributor Author

@imrannayer I have added that for safer_mysql too. I chose to make the variable default to true for safer_mysql, which I think is in-line with the philosophy of that module.

@OscarVanL OscarVanL changed the title feat: Support retain_backups_on_delete for mysql feat: Support retain_backups_on_delete for mysql, safer_mysql Jun 11, 2025
@imrannayer
Copy link
Collaborator

@OscarVanL thx. Can you plz fix lint issues.

@imrannayer imrannayer changed the title feat: Support retain_backups_on_delete for mysql, safer_mysql feat(TPG>6.31)!: Support retain_backups_on_delete for mysql, safer_mysql Jun 12, 2025
@OscarVanL
Copy link
Contributor Author

@imrannayer Oops, I've fixed the linting now.

@imrannayer
Copy link
Collaborator

/gcbrun

1 similar comment
@imrannayer
Copy link
Collaborator

/gcbrun

@imrannayer
Copy link
Collaborator

/gcbrun

@imrannayer imrannayer merged commit d0b31dd into terraform-google-modules:main Jun 17, 2025
4 checks passed
@OscarVanL
Copy link
Contributor Author

Thanks!

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.

2 participants