Skip to content

ci(release): skip the GHCR mirror when the chart version is already there - #172

Merged
jbouder merged 1 commit into
mainfrom
ci/skip-existing-chart-mirror
Oct 22, 2026
Merged

jbouder merged 1 commit into
mainfrom
ci/skip-existing-chart-mirror

Conversation

@jbouder

@jbouder jbouder commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Merging #171 touched charts/nebari-apps/Chart.yaml without a version bump, so Release ran. pack-release skipped the existing nebari-apps-0.1.0, but the Mirror chart to GHCR job pushed the same tarball again and moved the 0.1.0 tag to a new digest (sha256:0b38f02c… → sha256:496488f3…; the content is the released tarball, but helm push stamps a new manifest).

The mirror job now checks GHCR first (helm show chart … --version <version>, after the registry login) and skips the push when the version is already there. Any other failure of that check still pushes, so a real release never loses its mirror.

pack-release at the pinned commit exposes no outputs, so the job can't use the release job's result instead.

Checked: helm show chart against GHCR exits 0 for 0.1.0 and 1 for 9.9.9; actionlint 1.7.12 passes on release.yaml.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Ap1Tqx1QLHRGVVSXdxeduM

…here

A Chart.yaml change without a version bump (#171 removing the
release-please markers) re-ran Release: pack-release skipped the existing
nebari-apps-0.1.0, but mirror-oci pushed the same tarball again and moved
the 0.1.0 tag in GHCR to a new digest. pack-release has no output saying
it skipped, so the mirror job asks GHCR itself.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ap1Tqx1QLHRGVVSXdxeduM
@jbouder
jbouder merged commit ac0eacd into main Oct 22, 2026
8 checks passed
@jbouder
jbouder deleted the ci/skip-existing-chart-mirror branch October 22, 2026 13:29
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.

1 participant