fix(ci): stamp helmchart.yaml chartVersion at release time#24
Merged
Conversation
The HelmChart manifest must have a chartVersion matching the packaged archive. Replicated cannot match them when the field is empty, causing helm-archive-missing and helm-chart-missing errors. After copying helmchart.yaml to release/, use sed to replace the empty chartVersion with the build version before the release is created. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…yaml The helm-charts section was generating a conflicting HelmChart CR with no chartVersion, causing the helm-archive-missing errors. helmchart.yaml (kots.io/v1beta2) is now the explicit source of truth for the chart CR. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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
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.
Summary
Fixes two errors caused by a
chartVersionmismatch:Could not find helm archive for chart 'gameshelf' version ''Could not find helm chart manifest for archive 'gameshelf-0.1.43.tgz'Root cause: Two things were wrong:
helmchart.yamlshipped withchartVersion: "". Replicated requires this to exactly match the packaged.tgzarchive version (per the HelmChart v2 docs), but the version is only known at CI time..replicatedhad ahelm-chartssection that was generating a conflicting HelmChart CR with no version, which was the original source of the mismatch.Fix:
helm-chartsfrom.replicated—helmchart.yamlis now the explicit HelmChart CRhelmchart.yamltorelease/, stamp the correct version in withsed:release.ymluses${{ env.VERSION }}(e.g.0.1.43)pr.ymluses${{ env.CHART_VERSION }}(e.g.0.0.43)🤖 Generated with Claude Code