Skip to content

Ga security policy v2#25775

Merged
modular-magician merged 1 commit intohashicorp:mainfrom
modular-magician:downstream-pr-ea301692d7698e6f992b6395c7a4ab5690cda3b4
Jan 14, 2026
Merged

Ga security policy v2#25775
modular-magician merged 1 commit intohashicorp:mainfrom
modular-magician:downstream-pr-ea301692d7698e6f992b6395c7a4ab5690cda3b4

Conversation

@modular-magician
Copy link
Collaborator

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)

Derived from GoogleCloudPlatform/magic-modules#16094

[upstream:ea301692d7698e6f992b6395c7a4ab5690cda3b4]

Signed-off-by: Modular Magician <magic-modules@google.com>
@modular-magician modular-magician merged commit 04c6111 into hashicorp:main Jan 14, 2026
4 checks passed
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