close
Skip to content

Fix ComputeBackendService with INTERNAL_SELF_MANAGED loadBalancingScheme - #10414

Merged
gemmahou merged 1 commit into
GoogleCloudPlatform:masterfrom
codebot-robot:issue_10402
Jun 18, 2026
Merged

Fix ComputeBackendService with INTERNAL_SELF_MANAGED loadBalancingScheme#10414
gemmahou merged 1 commit into
GoogleCloudPlatform:masterfrom
codebot-robot:issue_10402

Conversation

@codebot-robot

Copy link
Copy Markdown
Collaborator

Prevent the automatic injection of subsetting: { policy: "NONE" } for regional backend services with INTERNAL_SELF_MANAGED load balancing scheme. This occurs because Traffic Director / INTERNAL_SELF_MANAGED schemes do not support subsetting, causing GCP API server validation to fail during creation.

I have verified the fix with a new E2E test case under pkg/test/resourcefixture/testdata/basic/compute/v1beta1/computebackendservice/regionalcomputebackendservicetd that runs successfully against MockGCP.

Fixes #10402

gemmahou
gemmahou previously approved these changes Jun 17, 2026
@gemmahou gemmahou added this to the 1.154.0 milestone Jun 17, 2026
@gemmahou
gemmahou enabled auto-merge June 17, 2026 22:27
@google-oss-prow google-oss-prow Bot added the lgtm label Jun 17, 2026
@gemmahou

Copy link
Copy Markdown
Collaborator

Test against realGCP and the resource with loadBalancingScheme: INTERNAL_SELF_MANAGED can be created successfully.

@gemmahou

Copy link
Copy Markdown
Collaborator

Please fix the failed ci-presubmit / tests-e2e-fixtures-compute (pull_request) job.

@gemmahou gemmahou assigned codebot-robot and unassigned gemmahou Jun 17, 2026
… loadBalancingScheme

Prevent the automatic injection of `subsetting: { policy: "NONE" }` for regional backend services with `INTERNAL_SELF_MANAGED` load balancing scheme. This is because subsetting is not supported for Traffic Director / INTERNAL_SELF_MANAGED scheme.

Fixes GoogleCloudPlatform#10402
auto-merge was automatically disabled June 17, 2026 22:44

Head branch was pushed to by a user without write access

@google-oss-prow google-oss-prow Bot removed the lgtm label Jun 17, 2026
@codebot-robot codebot-robot removed their assignment Jun 17, 2026
@gemmahou
gemmahou enabled auto-merge June 17, 2026 23:23
@google-oss-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: gemmahou
Once this PR has been reviewed and has the lgtm label, please assign fedebongio for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gemmahou
gemmahou added this pull request to the merge queue Jun 17, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 18, 2026
@gemmahou
gemmahou added this pull request to the merge queue Jun 18, 2026
Merged via the queue into GoogleCloudPlatform:master with commit 209cac7 Jun 18, 2026
184 of 355 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: ComputeBackendService fails to create with INTERNAL_SELF_MANAGED loadBalancingScheme due to auto-injected subsetting NONE

2 participants