Ga security policy v2

[SCSAndre]

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 (ga)

[github-actions]

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.

[maxi-cit]

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!

[github-actions]

@roaks3 This PR has been waiting for review for 3 weekdays. Please take a look! Use the label disable-review-reminders to disable these notifications.

[SCSAndre]

Hey @roaks3, could you take a look on that, please?

[roaks3]

LGTM; running tests now, although they will likely be unchanged because we only run beta tests

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

google provider: Diff ( 3 files changed, 48 insertions(+), 11 deletions(-))
google-beta provider: Diff ( 3 files changed, 2 insertions(+), 4 deletions(-))

[modular-magician]

Tests analytics

Total tests: 1325
Passed tests: 1238
Skipped tests: 86
Affected tests: 1

Click here to see the affected service packages

• compute

Action taken

Found 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests

• TestAccComputeSecurityPolicy_withAdvancedOptionsConfig

Get to know how VCR tests work

[modular-magician]

🟢 Tests passed during RECORDING mode:
TestAccComputeSecurityPolicy_withAdvancedOptionsConfig [Debug log]

🟢 No issues found for passed tests after REPLAYING rerun.

---

🟢 All tests passed!

View the build log or the debug log for each test

[SCSAndre]

@roaks3 could you, please, run the check again?

I forgot to change the meta.yaml file before.

Probably will pass now and can be merged.

Tks!

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

google provider: Diff ( 4 files changed, 50 insertions(+), 12 deletions(-))
google-beta provider: Diff ( 4 files changed, 3 insertions(+), 5 deletions(-))

[modular-magician]

Tests analytics

Total tests: 1325
Passed tests: 1239
Skipped tests: 86
Affected tests: 0

Click here to see the affected service packages

• compute

🟢 All tests passed!

View the build log

[SCSAndre]

@roaks3 tks for your attention Ryan!

Could you give me an estimate of when the merger will take place?

[SCSAndre]

@roaks3 tks a lot!