Skip to content

Conversation

@SCSAndre
Copy link

@SCSAndre SCSAndre commented Jan 7, 2026

Description

This PR promotes the request_body_inspection_size field from Beta to GA in the google_compute_security_policy resource.

GA API Confirmation: The Compute v1 (GA) API already exposes requestBodyInspectionSize and supports values up to 64KB.

Changes Made

1. Schema Definition

  • Removed version guards ({{- if ne $.TargetVersionName "ga" }}) from request_body_inspection_size field

2. Expand/Flatten Functions

  • Removed version guards from expandSecurityPolicyAdvancedOptionsConfig
  • Removed version guards from flattenSecurityPolicyAdvancedOptionsConfig

3. ForceSendFields

  • Updated to include advancedOptionsConfig.requestBodyInspectionSize for all versions

4. Documentation

  • Removed [Beta] marker from request_body_inspection_size field

5. Tests

  • Removed version guards from test steps for request_body_inspection_size
  • Tests now run for both GA and Beta providers

Testing

The acceptance test TestAccComputeSecurityPolicy_withAdvancedOptionsConfig includes:

  • Test step with request_body_inspection_size = "64KB"
  • Import verification
  • Full lifecycle validation
compute: promoted `request_body_inspection_size` to GA in `google_compute_security_policy` resource

@modular-magician modular-magician added the awaiting-approval Pull requests that need reviewer's approval to run presubmit tests label Jan 7, 2026
@github-actions github-actions bot requested a review from roaks3 January 7, 2026 21:32
@github-actions
Copy link

github-actions bot commented Jan 7, 2026

Hello! I am a robot. Tests will require approval from a repository maintainer to run.

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

@roaks3, 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.

…ompute_security_policy

This change promotes the request_body_inspection_size field from Beta to GA
in the google_compute_security_policy resource.

The GA Compute API (v1) now supports this field with values: 8KB, 16KB, 32KB,
48KB, and 64KB for WAF request body inspection configuration.

Changes:
- Removed version guards from schema definition
- Removed version guards from expand/flatten functions
- Updated ForceSendFields to include field for all versions
- Removed [Beta] marker from documentation
- Enabled tests for GA provider

```release-note:enhancement
compute: promoted `request_body_inspection_size` to GA in `google_compute_security_policy`
```
@SCSAndre SCSAndre force-pushed the ga-security-policy-v2 branch from 4bf5ecb to 74aca0d Compare January 7, 2026 22:10
@maxi-cit
Copy link
Contributor

maxi-cit commented Jan 8, 2026

Hi @roaks3, a client has asked about support related to this PR.
Would you mind running the tests to verify everything is in good shape?
Appreciate it!

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.

3 participants