Skip to content

Implement round-trip KRM fuzzer for NetworkServicesHTTPRoute#11744

Merged
barney-s merged 1 commit into
GoogleCloudPlatform:masterfrom
lovelace-coder-bot:issue-11743-1784452080
Jul 19, 2026
Merged

Implement round-trip KRM fuzzer for NetworkServicesHTTPRoute#11744
barney-s merged 1 commit into
GoogleCloudPlatform:masterfrom
lovelace-coder-bot:issue-11743-1784452080

Conversation

@lovelace-coder-bot

Copy link
Copy Markdown
Collaborator

This PR implements a lossless round-trip KRM fuzz test for the NetworkServicesHTTPRoute direct controller resource.

Changes:

  • Created pkg/controller/direct/networkservices/networkserviceshttproute_fuzzer.go implementing a protobuf-based KRMFuzzer.
  • Created custom HttprouteQueryParameters and HttprouteHeaders FromProto/ToProto mappers in networkserviceshttproute_mapper.go to handle PresentMatch oneof boolean fields correctly, preventing zero-value mapping loss.
  • Regenerated mappers using apis/networkservices/v1beta1/generate.sh.
  • Documented findings in .gemini/skills/create-fuzzer/journal/NetworkServicesHTTPRoute.md.

Fixes #11743

This PR was generated by the overseer,overseer agent (powered by the gemini-3.5-flash model).

This commit implements a lossless round-trip KRM fuzz test for the NetworkServicesHTTPRoute direct controller resource.

### Changes:
- Created pkg/controller/direct/networkservices/networkserviceshttproute_fuzzer.go implementing a protobuf-based KRMFuzzer.
- Created custom HttprouteQueryParameters and HttprouteHeaders FromProto/ToProto mappers in networkserviceshttproute_mapper.go to handle PresentMatch oneof boolean fields correctly, preventing zero-value mapping loss.
- Regenerated mappers using apis/networkservices/v1beta1/generate.sh.
- Documented findings in .gemini/skills/create-fuzzer/journal/NetworkServicesHTTPRoute.md.

Issue: 11743

@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

@barney-s
barney-s added this pull request to the merge queue Jul 19, 2026
@google-oss-prow google-oss-prow Bot added the lgtm label Jul 19, 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 cb97c86 Jul 19, 2026
202 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement round-trip KRM fuzzer for NetworkServicesHTTPRoute

2 participants