Skip to content

Remove retired docs E2E test scaffolding - #6751

Merged
knative-prow[bot] merged 1 commit into
knative:mainfrom
dprotaso:remove-unused-docs-e2e
Oct 27, 2026
Merged

knative-prow[bot] merged 1 commit into
knative:mainfrom
dprotaso:remove-unused-docs-e2e

Conversation

@dprotaso

Copy link
Copy Markdown
Member

Summary

  • Delete the unused test/e2e-tests.sh runner and empty test/e2e package.
  • Remove obsolete GKE/Istio/Knative Build instructions from test/README.md; document the remaining Go tests.
  • Keep existing unit tests, presubmit helpers, and vendored tooling unchanged.

The integration tests were disabled in #4300 and their Prow job was removed in knative/test-infra#2891. Current knative/infra job configurations contain no docs integration job.

Independent cleanup identified during review of #6748; this PR contains no sample modernization changes.

Validation

  • go test -mod=vendor ./... passed.
  • No remaining project references to the deleted runner/package (excluding generic vendored helpers).
  • git diff --check passed.

@knative-prow knative-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 27, 2026
@netlify

netlify Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for knative ready!

Name Link
🔨 Latest commit 09dc973
🔍 Latest deploy log https://app.netlify.com/projects/knative/deploys/6ae00b9aeee2574ccfd5a417
😎 Deploy Preview https://deploy-preview-6751--knative.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@knative-prow knative-prow Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 27, 2026
@dprotaso

Copy link
Copy Markdown
Member Author

/assign @dsimansk

@knative-prow knative-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Oct 27, 2026
@knative-prow

knative-prow Bot commented Oct 27, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprotaso, evankanderson

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

@knative-prow
knative-prow Bot merged commit dbb2647 into knative:main Oct 27, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants