Visitar URL original
fix dead Kubernetes link in job status reader by gerardrecinto · Pull Request #32729 · helm/helm · GitHub
Skip to content

fix dead Kubernetes link in job status reader - #32729

Merged
TerryHowe merged 1 commit into
helm:mainfrom
gerardrecinto:statusreader-job-link
Oct 18, 2026
Merged

TerryHowe merged 1 commit into
helm:mainfrom
gerardrecinto:statusreader-job-link

Conversation

@gerardrecinto

Copy link
Copy Markdown
Contributor

The comment in internal/statusreaders/job_status_reader.go points at pkg/controller/job/utils.go in kubernetes/kubernetes, which now 404s. The code lives at pkg/controller/job/util/utils.go, so this updates the link. I dropped the #L24 anchor since the line numbers there have moved. Comment-only change.

Thanks, Gerard Recinto

@pull-request-size pull-request-size Bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 6, 2026
@gerardrecinto

Copy link
Copy Markdown
Contributor Author

@TerryHowe @mattfarina @banjoh could one of you take a look when you have a moment? Comment-only fix for a dead link.

Thanks, Gerard Recinto

@github-actions github-actions Bot added the v4.x Issues and Pull Requests related to the major version v4 label Oct 6, 2026
@gerardrecinto

Copy link
Copy Markdown
Contributor Author

@banjoh @TerryHowe, thanks for the approval; build-test, CodeQL, and golangci-lint still show action_required. Could you approve the workflow runs or let me know the next step?

Thanks
Gerard Recinto

@TerryHowe TerryHowe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@TerryHowe TerryHowe added the Has One Approval This PR has one approval. It still needs a second approval to be merged. label Oct 18, 2026
Signed-off-by: Gerard Louis Recinto <gerardrecinto@gmail.com>
@TerryHowe
TerryHowe force-pushed the statusreader-job-link branch from 9704ed8 to 2a2c5bf Compare October 18, 2026 15:30
@TerryHowe
TerryHowe merged commit 30fd975 into helm:main Oct 18, 2026
6 checks passed
biagiopietro pushed a commit to biagiopietro/helm that referenced this pull request Oct 22, 2026
Signed-off-by: Gerard Louis Recinto <gerardrecinto@gmail.com>
@scottrigby scottrigby added docs and removed Has One Approval This PR has one approval. It still needs a second approval to be merged. labels Nov 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. v4.x Issues and Pull Requests related to the major version v4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants