Downloads: link the Helm chart tarball on the Kubernetes Helm card - #913
Merged
Conversation
skywalking-helm-<version>.tgz — the packaged chart, and the artifact people actually install — is published to the ASF dist but was never linked. The Kubernetes Helm card carried a Source section alone, offering only the source tarball, so the chart was reachable only by browsing the mirror by hand. Adds a Distribution section with the chart for 5.0.0 (closer.cgi plus downloads.apache.org signatures) and 4.9.0 (archive), labelled "Helm chart" to match what the SWCK card already shows. All six links verified to return 200. This is a standing omission rather than something 5.0.0 introduced — 4.9.0 shipped the chart too. It became visible now because SWCK 0.11.0 started publishing its own chart in #912, leaving two adjacent cards disagreeing about whether a released chart gets a link.
✅ Deploy Preview for skywalking-website-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Apache publishes two files for each Helm chart release. The downloads page linked only one.
The second one is the packaged chart — what
helm installactually consumes. With no link for it, the Kubernetes Helm card rendered a Source section and nothing else, and the chart was reachable only by browsing the ASF mirror by hand.Change
Adds a Distribution section to the Kubernetes Helm card carrying the chart for:
closer.cgifor the download,downloads.apache.orgfor.asc/.sha512archive.apache.orgthroughoutlabelled
Helm chart, matching what the SkyWalking Cloud on Kubernetes card already shows.Why it went unnoticed
Not a 5.0.0 regression — 4.9.0 ships a chart too, with full signatures, and was equally unlinked. It only became visible because SWCK 0.11.0 began publishing its own chart in #912, which left two adjacent cards disagreeing about whether a released chart gets a link.
For reference, the three chart artifacts on the site now behave consistently, and the differences between them are deliberate:
apache/skywalking-helmapache/skywalking-swck, new in 0.11.0apache/skywalking-banyandb-helm, a separate projectVerification
All six new links probed — every one returns 200, across both the current-release and archive paths. Full Hugo build: the card renders
Source → srcandDistribution → Helm chart, with 4.9.0 behind "+1 more" in both sections.