Skip to content

Add KIND-backed JSON and protobuf end-to-end tests - #3135

Draft
brendandburns with Copilot wants to merge 1 commit into
mainfrom
copilot/kind-e2e-protobuf
Draft

brendandburns with Copilot wants to merge 1 commit into
mainfrom
copilot/kind-e2e-protobuf

Conversation

Copilot AI commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add npm run e2e-test covering ConfigMap CRUD through the generated JSON client and kubeconfig-authenticated protobuf requests.
  • Validate protobuf content types, Kubernetes binary envelopes, decoded data, and JSON interoperability, with isolated namespace cleanup.
  • Run the suite in existing KIND-backed CI and a new Copilot setup workflow that leaves its KIND cluster available to the agent.
  • Document local KIND setup and test usage.

Validation

  • Both end-to-end tests passed against KIND with Kubernetes v1.35.0.
  • npm run build-with-tests and npm run lint passed.
  • Workflow YAML parsing and git diff --check passed.

The protobuf tests exercise binary requests directly; the generated API client remains JSON-based.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@kubernetes-prow kubernetes-prow Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 4, 2026
@kubernetes-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Copilot
Once this PR has been reviewed and has the lgtm label, please ask for approval from brendandburns. For more information see the 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

@linux-foundation-easycla

Copy link
Copy Markdown

CLA Not Signed

@kubernetes-prow kubernetes-prow Bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Nov 4, 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

cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants