Skip to content

docs: Fix 404 helm-charts links on Platform Helm page [EDU-1330] - #1706

Merged
justinegeffen merged 1 commit into
masterfrom
edu-1330-fix-helm-chart-links
Aug 3, 2026
Merged

docs: Fix 404 helm-charts links on Platform Helm page [EDU-1330]#1706
justinegeffen merged 1 commit into
masterfrom
edu-1330-fix-helm-chart-links

Conversation

@robsyme

@robsyme robsyme commented Aug 3, 2026

Copy link
Copy Markdown
Member

Fixes EDU-1330. Reported by a customer via FD-7794.

Problem

All four seqeralabs/helm-charts links on the Platform Enterprise Helm installation page 404, in every doc version that carries the page (next, 26.1, 25.3, 25.2, 25.1) — 20 dead links. They've been broken since 2026-07-20.

Two independent upstream changes compounded:

  1. Tag rename. helm-charts renamed its bare-numeric tags to <chart>-<version>. Of 242 tags, only 0.16.1 is still un-prefixed, so /tree/0.36.1/... can never resolve — and GitHub doesn't redirect a missing ref, so it fails hard.
  2. Monorepo restructure. The platform chart moved from platform/ to charts/platform/ between platform-0.20.1 and platform-0.30.0. This is why even the next docs, which did carry the platform- prefix, still 404.

#1660 substituted 0.20.10.36.1 in the URLs and changed nothing else, inheriting both the missing prefix and the pre-monorepo path. It also preserved a pre-existing inconsistency — the next copy used platform-0.20.1 while the versioned copies used bare 0.20.1 — so after the bump the two sets fail for different reasons.

Change

Adds the tag prefix and the charts/ path segment:

- https://github.com/seqeralabs/helm-charts/blob/0.36.1/platform/examples/kustomize/values.yaml
+ https://github.com/seqeralabs/helm-charts/blob/platform-0.36.1/charts/platform/examples/kustomize/values.yaml

The studios and pipeline-optimization links are deliberately untouched — those charts kept the platform/charts/<name> layout and already resolve, so a repo-wide sweep would have broken them.

Verification

Status-checked every unique helm-charts URL in the repo before and after. 6 of 15 were 404; all 12 distinct URLs now return 200.

grep -rhno "https://github.com/seqeralabs/helm-charts/[a-zA-Z0-9./_-]*" --exclude-dir=.git . \
  | sed 's/^[0-9]*://' | sort -u \
  | while read -r u; do echo "$(curl -s -o /dev/null -w '%{http_code}' "$u")  $u"; done | sort

The one remaining 301 is GitHub redirecting treeblob for examples/seqera-ai, which is a file. Pre-existing and harmless.

Why CI didn't catch this

Worth flagging separately, because it's the more interesting finding. links.yml runs lychee over the docs, but its last output was #440 on 2025-02-08 and all 218 runs in the retained history failed. Three independent defects:

  • No pull_request trigger. Triggers are schedule (Sunday 18:00), workflow_dispatch, repository_dispatch. Even a working checker could not have blocked docs: Update platform helm chart to latest #1660 — at best it would have complained the following Sunday. Only check-internal-links.yml gates PRs, and that covers internal links only.
  • lychee rejects its own arguments. --base . has been there since the workflow was written in July 2024, but lychee 2.x requires an absolute path or URL and moved root-relative local resolution to --root-dir. The binary exits 2 after ~200ms without checking a link. Because that's an arg-parse error rather than a link failure, steps.lychee.outputs.exit_code is never set, so create-issue-from-file is skipped too — no report issue, no signal at all.
  • The glob misses 94% of the docs. Args end in './**/*.mdx'; the enterprise docs are 1284 .md against 75 .mdx. platform-helm.md was never in scope.

Fixing all three (add pull_request, swap --base . for --root-dir "$GITHUB_WORKSPACE", glob './**/*.md*') is what would actually catch the next one. Not done here — it's a separate change, and the first successful run will surface 18 months of accumulated rot, so it'll likely need an allowlist or a soft-fail period before it can gate PRs.

Also not addressed

Editorial, tracked on EDU-1330:

  • The chart version is copy-pasted into four URLs across five files. A Docusaurus constant would make the bump a one-liner.
  • The 25.1/25.2/25.3 pages point at chart 0.36.1 (appVersion v26.1.3), which postdates those releases.
  • 0.36.1 is already behind — 0.37.0 and 0.38.0 (appVersion v26.1.4) exist.

🤖 Generated with Claude Code

All four helm-charts links on the Platform Enterprise Helm installation
page 404, across all five doc versions that carry it (next, 26.1, 25.3,
25.2, 25.1) — 20 links in total.

Two independent upstream changes in seqeralabs/helm-charts compounded:

1. Bare-numeric tags were renamed to <chart>-<version>. Only 0.16.1 is
   still un-prefixed, so /tree/0.36.1/... can never resolve, and GitHub
   does not redirect a missing ref.
2. The platform chart moved from platform/ to charts/platform/ when the
   repo became a multi-chart monorepo, between 0.20.1 and 0.30.0. This
   is why even the next docs — which did carry the platform- prefix —
   still 404.

PR #1660 substituted 0.20.1 -> 0.36.1 in the URLs and changed nothing
else, inheriting both the missing prefix and the pre-monorepo path.

Adds the tag prefix and the charts/ segment. The studios and
pipeline-optimization links are deliberately untouched: those charts
kept the platform/charts/<name> layout and already resolve.

Reported by a customer via FD-7794.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Signed-off-by: Rob Syme <rob.syme@gmail.com>
@netlify

netlify Bot commented Aug 3, 2026

Copy link
Copy Markdown

Deploy Preview for seqera-docs ready!

Name Link
🔨 Latest commit debbaee
🔍 Latest deploy log https://app.netlify.com/projects/seqera-docs/deploys/6a70ca0d0dc00b0008f78996
😎 Deploy Preview https://deploy-preview-1706--seqera-docs.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.

@justinegeffen justinegeffen added the 2. Edu reviews complete Reviews complete. Remove label when confirmed in prod. label Aug 3, 2026
@justinegeffen
justinegeffen merged commit 3601366 into master Aug 3, 2026
10 checks passed
@justinegeffen
justinegeffen deleted the edu-1330-fix-helm-chart-links branch August 3, 2026 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. Edu reviews complete Reviews complete. Remove label when confirmed in prod.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants