Skip to content

Fixed Invalid value for field rateLimitOptions when updating RegionSecurityPolicy rules#16757

Open
matheusaleixo-cit wants to merge 2 commits intoGoogleCloudPlatform:mainfrom
matheusaleixo-cit:fix-invalid-value-for-rate-limit-options-when-updating-compute-region-security-policy-rule
Open

Fixed Invalid value for field rateLimitOptions when updating RegionSecurityPolicy rules#16757
matheusaleixo-cit wants to merge 2 commits intoGoogleCloudPlatform:mainfrom
matheusaleixo-cit:fix-invalid-value-for-rate-limit-options-when-updating-compute-region-security-policy-rule

Conversation

@matheusaleixo-cit
Copy link
Contributor

Updates use of updateMask for rate_limit_options in google_compute_region_security_policy_rule resources.
Also adds tests to replicate the reported issue.

Similar to the fix for global policies from this PR.

Part of: #17275

Release Note Template for Downstream PRs (will be copied)

See Write release notes for guidance.

compute: updated request updateMask for `rate_limit_options` field  to `google_compute_region_security_policy_rule` resource

@modular-magician modular-magician added the awaiting-approval Pull requests that need reviewer's approval to run presubmit tests label Mar 18, 2026
@matheusaleixo-cit matheusaleixo-cit marked this pull request as ready for review March 19, 2026 22:55
@github-actions github-actions bot requested a review from zli82016 March 19, 2026 22:55
@github-actions
Copy link

Googlers: For automatic test runs see go/terraform-auto-test-runs.

@zli82016, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look.

You can help make sure that review is quick by doing a self-review and by running impacted tests locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-approval Pull requests that need reviewer's approval to run presubmit tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants