Skip to content

Feat/cloudquotas ignore safety checks set - #1

Open
mfreymuth wants to merge 2 commits into
mainfrom
feat/cloudquotas-ignore-safety-checks-set
Open

Feat/cloudquotas ignore safety checks set#1
mfreymuth wants to merge 2 commits into
mainfrom
feat/cloudquotas-ignore-safety-checks-set

Conversation

@mfreymuth

Copy link
Copy Markdown
Owner

Release Note Template for Downstream PRs (will be copied)

See Write release notes for guidance.


…ead of URL templates

The ignoreSafetyChecks query parameter was interpolated directly in
create_url and update_url, which only supports a single scalar value.
Move its construction to pre_create/pre_update custom code so it can
later support the repeated form accepted by the API. No behavior
change: the parameter is now omitted when the field is unset or
QUOTA_SAFETY_CHECK_UNSPECIFIED, which the API treats identically.
@mfreymuth
mfreymuth force-pushed the feat/cloudquotas-ignore-safety-checks-set branch 2 times, most recently from b105947 to 24cb223 Compare August 17, 2026 22:13
…deprecate ignore_safety_checks

The QuotaPreferences API declares ignoreSafetyChecks as a repeated
enum query parameter, but the provider modeled it as a single string.
When a change trips both QUOTA_DECREASE_BELOW_USAGE and
QUOTA_DECREASE_PERCENTAGE_TOO_HIGH (for example lowering a quota to 0
with recent usage), no single value works and the change cannot be
applied with Terraform at all.

Add ignore_safety_checks_set, a set of enum values sent as a repeated
query parameter, and deprecate ignore_safety_checks. The mutual
exclusion is enforced with a CustomizeDiff because ConflictsWith is
not generated for url_param_only parameters.

Fixes hashicorp/terraform-provider-google#19276
Fixes hashicorp/terraform-provider-google#25454
@mfreymuth
mfreymuth force-pushed the feat/cloudquotas-ignore-safety-checks-set branch from 24cb223 to f612792 Compare August 17, 2026 22:43
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.

1 participant