Skip to content

Greenfield: Align MockGCP logs with RealGCP for MapManagementStyleConfig - #13638

Open
neumann-coder-bot wants to merge 1 commit into
GoogleCloudPlatform:masterfrom
neumann-coder-bot:issue-13624-1790901405
Open

neumann-coder-bot wants to merge 1 commit into
GoogleCloudPlatform:masterfrom
neumann-coder-bot:issue-13624-1790901405

Conversation

@neumann-coder-bot

Copy link
Copy Markdown
Collaborator

Overview

This PR verifies and confirms the alignment of MockGCP logs with RealGCP output for MapManagementStyleConfig (mapmanagement.cnrm.cloud.google.com/v1alpha1) in accordance with the Greenfield MockGCP alignment workflow.

Changes

  • Added unit tests for MockGCP MapManagementStyleConfig in mockgcp/mockmapmanagement/styleconfig_test.go covering resource name parsing and CRUD operations (CreateStyleConfig, GetStyleConfig, UpdateStyleConfig, DeleteStyleConfig, ListStyleConfigs).
  • Verified golden log alignment (_http.log vs _http_mock.log) for both mapmanagementstyleconfig-minimal and mapmanagementstyleconfig-maximal test fixtures.
  • Verified e2e tests passing against MockGCP with re-reconciliation and deletion defense.

Fixes #13624

NONE

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

…nfig

Adds unit tests for MockGCP MapManagementStyleConfig CRUD operations and name parsing.
Verifies golden log alignment against real GCP.

Fix: GoogleCloudPlatform#13624

@anhdle-sso anhdle-sso 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

@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 NOT APPROVED

This pull-request has been approved by: anhdle-sso
Once this PR has been reviewed and has the lgtm label, please assign barney-s 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

@anhdle-sso
anhdle-sso added this pull request to the merge queue Oct 2, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 2, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

3 participants