Skip to content

Add missing deletion_protection variable for terraform resource#1909

Merged
jcscottiii merged 1 commit intomainfrom
jcscottiii/add-protection-deletion
Oct 8, 2025
Merged

Add missing deletion_protection variable for terraform resource#1909
jcscottiii merged 1 commit intomainfrom
jcscottiii/add-protection-deletion

Conversation

@jcscottiii
Copy link
Collaborator

@jcscottiii jcscottiii commented Oct 8, 2025

Some GCP terraform resources have an optional property named deletion_protection.

With it, terraform will not delete it. It defaults to true for staging and prod. But false when deploying your own copy.

When I was cleaning up old dev environments, I could not cleanly delete them because this particular resource had deletion_protection set to true by default. This change uses the global variable which allows us to explicitly tell if we want that given resource to have deletion_protection.

Some GCP terraform resources have an optional property named deletion_protection.

With it, terraform will not delete it. It defaults to true for staging and prod.\
But false when deploying your own [copy](https://github.com/GoogleChrome/webstatus.dev/blob/main/DEPLOYMENT.md#deploying-your-own-copy).

When I was cleaning up old dev environments, I could not cleanly delete them because this particular resource
had deletion_protection set to true by default. This change uses the global variable which allows us to explicitly tell if we want
that given resource to have deletion_protection.
@jcscottiii jcscottiii enabled auto-merge October 8, 2025 18:11
@jcscottiii jcscottiii added this pull request to the merge queue Oct 8, 2025
Merged via the queue into main with commit 7342afa Oct 8, 2025
7 checks passed
@jcscottiii jcscottiii deleted the jcscottiii/add-protection-deletion branch October 8, 2025 18:45
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