Repository navigation
ci: automate Helm chart sync to fluent/helm-charts - #2092
Draft
joshuabaird wants to merge 1 commit into
Draft
joshuabaird wants to merge 1 commit into
joshuabaird wants to merge 1 commit into
Conversation
Adds hack/sync-helm-charts.sh and a sync-helm-charts workflow that release-please dispatches after each release. It opens a PR in fluent/helm-charts with the synced charts, bumped versions, generated CHANGELOG entries and regenerated READMEs. Removes the stale chart CHANGELOGs here; fluent/helm-charts owns them. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: Josh Baird <jbaird@galileo.io>
Collaborator
Author
|
@edsiper Could you help us get a Github App created for this automation? Thanks! |
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.
What this PR does / why we need it:
Publishing the charts to fluent/helm-charts after each release is currently a manual copy-and-bump (e.g. fluent/helm-charts#767). This automates it:
hack/sync-helm-charts.sh <tag> <helm-charts-dir>copies the charts from the release tag, setsappVersion, bumps each chartversionby the same level as the operator release, adds CHANGELOG entries from conventional commits that touched each chart, and runs helm-docs.sync-helm-chartsworkflow runs the script and opens a PR in fluent/helm-charts. release-please dispatches it after each release, and it can be dispatched manually to re-run.Running the script for v3.11.0 against helm-charts before #767 produced the same charts, CRDs, values and READMEs as #767. Only the CHANGELOG wording differs, because the generated entries come from commit subjects.
Before this can merge:
contents: writeandpull-requests: write. Its client ID goes in theHELM_CHARTS_APP_CLIENT_IDvariable and its private key in theHELM_CHARTS_APP_PRIVATE_KEYsecret.Which issue(s) this PR fixes:
None
Does this PR introduced a user-facing change?
Additional documentation, usage docs, etc.: