Skip to content

Greenfield: Align MockGCP logs with RealGCP for NetworkSecurityMirroringEndpointGroupAssociation - #13663

Merged
barney-s merged 1 commit into
GoogleCloudPlatform:masterfrom
hopper-coder-bot:issue-13660-1790957963
Oct 2, 2026
Merged

barney-s merged 1 commit into
GoogleCloudPlatform:masterfrom
hopper-coder-bot:issue-13660-1790957963

Conversation

@hopper-coder-bot

Copy link
Copy Markdown
Collaborator

Description

This PR verifies and confirms that MockGCP HTTP logs for NetworkSecurityMirroringEndpointGroupAssociation are fully aligned with Real GCP golden logs (_http.log). All test fixtures (networksecuritymirroringendpointgroupassociation-minimal and networksecuritymirroringendpointgroupassociation-maximal) under pkg/test/resourcefixture/testdata/basic/networksecurity/v1alpha1/networksecuritymirroringendpointgroupassociation/ pass golden log alignment and E2E simulation tests. Added mocknetworksecurity service creation unit test in mockgcp/mocknetworksecurity/service_test.go.

Fixes #13660

NONE

This PR was generated by the overseer,step/mockgcp,greenfield agent (powered by the gemini-3.7-flash model).

@barney-s barney-s left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

harmless UT

@barney-s
barney-s added this pull request to the merge queue Oct 2, 2026
@google-oss-prow google-oss-prow Bot added the lgtm label Oct 2, 2026
@google-oss-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: barney-s

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

The pull request process is described 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

Merged via the queue into GoogleCloudPlatform:master with commit 26059f8 Oct 2, 2026
258 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved greenfield Indicates implementation of a new resource (vs migration) lgtm overseer/ready-for-human overseer step/mockgcp

3 participants