From 661d5787f6cbcebc1249640ec6babe73bc9e2334 Mon Sep 17 00:00:00 2001 From: Stefan Hausotte Date: Tue, 1 Sep 2026 18:39:09 +0200 Subject: [PATCH] ci: allow merging chart changes without a version bump `ct lint` enforced a version increment on every pull request touching the chart, so each merge had to ship a chart release. Disable the check. A merge without a bump must not break the release job either. chart-releaser picks changed charts by diffing the latest tag against HEAD rather than by comparing versions, so it repackages the unchanged version and aborts on the already existing release. Pass `skip_existing` so the upload is skipped, the published release stays untouched, and the changes ship with the next bump. --- .github/ct.yaml | 5 +++++ .github/workflows/ci.yaml | 7 +++++++ 2 files changed, 12 insertions(+) diff --git a/.github/ct.yaml b/.github/ct.yaml index 5f6be25..a7081a4 100644 --- a/.github/ct.yaml +++ b/.github/ct.yaml @@ -4,5 +4,10 @@ chart-dirs: - charts # The chart intentionally ships without a maintainers field; don't fail on it. validate-maintainers: false +# A pull request may change the chart without bumping its version. Such a change +# simply is not released: chart-releaser skips a version that already has a +# release, and the next version bump ships the accumulated changes. Requiring a +# bump per pull request would force a chart release for every merge. +check-version-increment: false # yamllint config matching the chart's inline-comment style. lint-conf: .github/lintconf.yaml diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index fd7ee28..512f0b2 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -106,7 +106,14 @@ jobs: with: version: v3.16.3 + # A merge that changes the chart without bumping its version must not fail + # here. chart-releaser detects the chart as changed since the last tag and + # repackages it, so without `skip_existing` the upload of an already + # released version aborts the job. Skipping leaves the existing release + # untouched; the changes ship with the next version bump. - name: Run chart-releaser uses: helm/chart-releaser-action@v1.7.0 + with: + skip_existing: true env: CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"