Skip to content

Automate release workflow#578

Merged
kd7lxl merged 2 commits intomasterfrom
release-workflow
Feb 18, 2026
Merged

Automate release workflow#578
kd7lxl merged 2 commits intomasterfrom
release-workflow

Conversation

@kd7lxl
Copy link
Collaborator

@kd7lxl kd7lxl commented Dec 9, 2025

What this PR does:
Automates the release workflow. It is manually triggered (for now) from Actions tab. There is a helper make bump-version that will look at the new entries in the changelog and apply semver rules to bump the version in Chart.yaml and CHANGELOG.md. Next it packages the chart, updates the index, and opens a PR with the changes.

Checklist

  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX], [DEPENDENCY]

Signed-off-by: Tom Hayward <thayward@infoblox.com>
@kd7lxl kd7lxl requested a review from nschad December 9, 2025 18:41
@nschad
Copy link
Collaborator

nschad commented Dec 17, 2025

This looks like the current "process" but honestly we should probably (if we're gonna switch). Just use github chart-releaser action. https://github.com/helm/chart-releaser

@kd7lxl
Copy link
Collaborator Author

kd7lxl commented Feb 17, 2026

This looks like the current "process" but honestly we should probably (if we're gonna switch). Just use github chart-releaser action. https://github.com/helm/chart-releaser

Looks like this action does not allow configuring the package-path to ./docs: https://github.com/helm/chart-releaser-action/blob/3e001cb8c68933439c7e721650f20a07a1a5c61e/cr.sh#L320. CLI will take precedence over any supplied config

# Update Changelog
date_str=$(date +%Y-%m-%d)
# Use perl for consistent behavior across platforms (macOS/Linux) regarding newlines in replacement
perl -i -pe "s/^## master \/ unreleased/## master \/ unreleased\n\n## $new_version \/ $date_str/" CHANGELOG.md
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is that installed in the job container?

Copy link
Collaborator

@nschad nschad left a comment

Choose a reason for hiding this comment

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

Looks ok.

I don't know to be honest. anyway we can test this?

@kd7lxl
Copy link
Collaborator Author

kd7lxl commented Feb 18, 2026

Looks ok.

I don't know to be honest. anyway we can test this?

It needs to be on the default branch before we can test it. Once it's there, we should be able to trigger versions of it on any branch, meaning if the first PR has a bug, we can test that the follow-up PR fixes it. Should take two PRs max to get working. 🤞

@kd7lxl kd7lxl enabled auto-merge (squash) February 18, 2026 22:00
@kd7lxl kd7lxl merged commit c0c0ce9 into master Feb 18, 2026
2 checks passed
@kd7lxl kd7lxl deleted the release-workflow branch February 18, 2026 22:59
@kd7lxl kd7lxl mentioned this pull request Feb 18, 2026
1 task
Calebjh pushed a commit to Calebjh/cortex-helm-chart that referenced this pull request Feb 19, 2026
Signed-off-by: Tom Hayward <thayward@infoblox.com>
Signed-off-by: chorst <chorst@infoblox.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments