Skip to content

docs: fix broken GKE setup link#3429

Open
leno23 wants to merge 1 commit into
tektoncd:mainfrom
leno23:fix-development-gke-link
Open

docs: fix broken GKE setup link#3429
leno23 wants to merge 1 commit into
tektoncd:mainfrom
leno23:fix-development-gke-link

Conversation

@leno23
Copy link
Copy Markdown

@leno23 leno23 commented May 18, 2026

Changes

  • replace the broken Knative GKE setup link in DEVELOPMENT.md with the current Google Cloud GKE quickstart

Fixes #1914

Validation

  • verified the replacement link returns HTTP 200

Signed-off-by: wuyangfan <yangfan.wu@succaiss.com>
@tekton-robot tekton-robot added the do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. label May 18, 2026
@linux-foundation-easycla
Copy link
Copy Markdown

CLA Missing ID

  • ❌ The email address for the commit (4e767bc) is not linked to the GitHub account, preventing the EasyCLA check. Consult this Help Article and GitHub Help to resolve. (To view the commit's email address, add .patch at the end of this PR page's URL.) For further assistance with EasyCLA, please visit our EasyCLA portal and chat with our support bot.

@tekton-robot tekton-robot requested a review from jkhelil May 18, 2026 03:10
@tekton-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign vdemeester after the PR has been reviewed.
You can assign the PR to them by writing /assign @vdemeester in a comment when ready.

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

@tekton-robot tekton-robot requested a review from mbpavan May 18, 2026 03:10
@tekton-robot tekton-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label May 18, 2026
@anithapriyanatarajan
Copy link
Copy Markdown
Contributor

@leno23 Thank you for the contribution! A couple of things to address before this can be merged:

  1. EasyCLA: The CLA check is failing because the email on commit 4e767bc isn't linked to your GitHub account. Please sign the EasyCLA and ensure your commit email matches your GitHub account — see the Tekton community CLA process and the GitHub help article for guidance.

  2. Scope of the fix: A simple link redirect to the generic GKE quickstart may not be sufficient here. The original link pointed to Knative-with-GKE.md, which captured a specific set of pre-requisite tooling (e.g. kubectl, gcloud, etc.) tailored to the Tekton operator setup. The current replacement doesn't cover those prerequisites. We should revisit what the right target content is, this likely warrants updating the DEVELOPMENT.md section itself rather than just swapping the URL.

@anithapriyanatarajan
Copy link
Copy Markdown
Contributor

/release-note-none

@tekton-robot tekton-robot added release-note-none Denotes a PR that doesnt merit a release note. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels May 18, 2026
@anithapriyanatarajan
Copy link
Copy Markdown
Contributor

/hold

@tekton-robot tekton-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. release-note-none Denotes a PR that doesnt merit a release note. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Link broken on tektoncd/operator/blob/main/DEVELOPMENT.md

3 participants