chore(chart): Refactor chart release logic to support releasing from master

stevehipwell · stevehipwell · commit c12fcb4234e1 · 2025-09-08T10:52:46.000+01:00
Signed-off-by: Steve Hipwell &lt;steve.hipwell@gmail.com&gt;
diff --git a/.github/workflows/lint-test-chart.yaml b/.github/workflows/lint-test-chart.yaml
@@ -2,9 +2,11 @@ name: Lint and Test Chart
 
 on:
   pull_request:
+    branches:
+      - master
+      - release-v*
     paths:
-      - .github/workflows/lint-test-chart.yaml
-      - "charts/metrics-server/**"
+      - charts/metrics-server/**
 
 permissions:
   contents: read
@@ -23,6 +25,23 @@ jobs:
         with:
           fetch-depth: 0
 
+      - name: Setup
+        id: setup
+        run: |
+          set -euo pipefail
+
+          chart_version="$(yq eval '.version' ./charts/metrics-server/Chart.yaml)"
+          changed=false
+
+          if [[ -n "$(ct list-changed --target-branch=master)" ]]; then
+            changed=true
+          fi
+
+          {
+            echo "chart_version=${chart_version}"
+            echo "changed=${changed}"
+          } >> "${GITHUB_OUTPUT}"
+
       - name: Set-up Python
         uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5.6.0
         with:
@@ -36,33 +55,15 @@ jobs:
       - name: Set-up chart-testing
         uses: helm/chart-testing-action@0d28d3144d3a25ea2cc349d6e59901c4ff469b3b # v2.7.0
 
-      - name: Check for changes
-        id: changes
-        run: |
-          changed="$(ct list-changed --target-branch=master)"
-          if [[ -n "${changed}" ]]
-          then
-            echo "changed=true" >> "${GITHUB_OUTPUT}"
-          else
-            echo "changed=false" >> "${GITHUB_OUTPUT}"
-          fi
-
-      - name: Get chart version
-        id: chart_version
-        if: steps.changes.outputs.changed == 'true'
-        uses: mikefarah/yq@f03c9dc599c37bfcaf533427211d05e51e6fee64 # v4.47.1
-        with:
-          cmd: yq eval '.version' './charts/metrics-server/Chart.yaml'
-
       - name: Get changelog entry
-        if: steps.changes.outputs.changed == 'true'
+        if: steps.setup.outputs.changed == 'true'
         uses: mindsers/changelog-reader-action@32aa5b4c155d76c94e4ec883a223c947b2f02656 # v2.2.3
         with:
           path: charts/metrics-server/CHANGELOG.md
-          version: ${{ steps.chart_version.outputs.result }}
+          version: ${{ steps.setup.outputs.chart_version }}
 
       - name: Set-up Artifact Hub CLI
-        if: steps.changes.outputs.changed == 'true'
+        if: steps.setup.outputs.changed == 'true'
         uses: action-stars/install-tool-from-github-release@f2e83e089fa618aa7e9fd3452fbcf4fe1598ede2 # v0.2.5
         with:
           github_token: ${{ github.token }}
@@ -73,21 +74,21 @@ jobs:
           version: latest
 
       - name: Run Artifact Hub lint
-        if: steps.changes.outputs.changed == 'true'
+        if: steps.setup.outputs.changed == 'true'
         run: ah lint --kind helm || exit 1
 
       - name: Run chart-testing lint
-        if: steps.changes.outputs.changed == 'true'
+        if: steps.setup.outputs.changed == 'true'
         run: ct lint --target-branch=master --check-version-increment=false
 
       - name: Create Kind cluster
-        if: steps.changes.outputs.changed == 'true'
+        if: steps.setup.outputs.changed == 'true'
         uses: helm/kind-action@a1b0e391336a6ee6713a0583f8c6240d70863de3 # v1.12.0
         with:
           wait: 120s
 
       - name: Install cert-manager dependency
-        if: steps.changes.outputs.changed == 'true'
+        if: steps.setup.outputs.changed == 'true'
         run: |
           helm repo add jetstack https://charts.jetstack.io
           helm install cert-manager jetstack/cert-manager \
@@ -98,7 +99,7 @@ jobs:
             --set extraArgs='{--enable-certificate-owner-ref}'
 
       - name: Prepare existing secret test scenario
-        if: steps.changes.outputs.changed == 'true'
+        if: steps.setup.outputs.changed == 'true'
         run: |
           openssl req -x509 -newkey rsa:2048 -sha256 -days 365 \
             -nodes -keyout ${{ runner.temp }}/tls.key -out ${{ runner.temp }}/tls.crt \
@@ -119,5 +120,5 @@ jobs:
           rm ${{ runner.temp }}/tls.key ${{ runner.temp }}/tls.crt
 
       - name: Run chart-testing install
-        if: steps.changes.outputs.changed == 'true'
+        if: steps.setup.outputs.changed == 'true'
         run: ct install --target-branch=master --namespace kube-system
diff --git a/.github/workflows/release-chart.yaml b/.github/workflows/release-chart.yaml
@@ -3,6 +3,7 @@ name: Release Chart
 on:
   push:
     branches:
+      - master
       - release-*
     paths:
       - charts/metrics-server/Chart.yaml
@@ -26,67 +27,59 @@ jobs:
         with:
           fetch-depth: 0
 
-      - name: Configure Git
-        run: |
-          git config user.name "$GITHUB_ACTOR"
-          git config user.email "$GITHUB_ACTOR@users.noreply.github.com"
-
-      - name: Get chart version
-        id: chart_version
-        uses: mikefarah/yq@f03c9dc599c37bfcaf533427211d05e51e6fee64 # v4.47.1
-        with:
-          cmd: yq eval '.version' './charts/metrics-server/Chart.yaml'
-
-      - name: Get chart app version
-        id: chart_app_version
-        uses: mikefarah/yq@f03c9dc599c37bfcaf533427211d05e51e6fee64 # v4.47.1
-        with:
-          cmd: yq eval '.appVersion' './charts/metrics-server/Chart.yaml'
-
-      - name: Check can release
-        id: check_can_release
+      - name: Setup
+        id: setup
         run: |
           set -euo pipefail
 
-          branch_name="${GITHUB_REF##*/}"
-          app_version_prefix="${branch_name##*-}"
+          chart_file="./charts/metrics-server/Chart.yaml"
 
-          app_version_regex="^${app_version_prefix//./\.}"
+          chart_version="$(yq eval '.version' "${chart_file}")"
+          app_version="$(yq eval '.appVersion' "${chart_file}")"
+          release=false
 
-          chart_version_match="$(echo "${{ steps.chart_version.outputs.result }}" | grep -Po "^\d+\.\d+\.\d+$" || true)"
-          app_version_match="$(echo "${{ steps.chart_app_version.outputs.result }}" | grep -Po "^${app_version_prefix//./\.}" || true)"
-
-          if [[ -z "${chart_version_match}" ]] || [[ -z "${app_version_match}" ]]
-          then
-            echo "continue=false" >> $GITHUB_OUTPUT
-          else
-            echo "continue=true" >> $GITHUB_OUTPUT
+          if [[ -z "$(gh release view --repo kubernetes-sigs/metrics-server "metrics-server-helm-chart-${chart_version}" --json id || true)" ]]; then
+            branch_name='${{ github.ref_name }}'
+            if [[ "${branch_name}" == "master" ]]; then
+              release=true
+            else
+              branch_version_prefix="$(grep -Eo '[0-9]+\.[0-9]+$' <<<"${branch_name}")"
+              if [[ -n "$(grep -o "${branch_version_prefix}" <<<"${app_version}" || true)" ]]; then
+                release=true
+              fi
+            fi
           fi
 
+          {
+            echo "chart_version=${chart_version}"
+            echo "app_version=${app_version}"
+            echo "release=${release}"
+          } >> "${GITHUB_OUTPUT}"
+
       - name: Set-up Helm
-        if: steps.check_can_release.outputs.continue == 'true'
+        if: steps.setup.outputs.release == 'true'
         uses: azure/setup-helm@b9e51907a09c216f16ebe8536097933489208112 # v4.3.0
         with:
           version: latest
 
       - name: Get CHANGELOG entry
         id: changelog_reader
-        if: steps.check_can_release.outputs.continue == 'true'
+        if: steps.setup.outputs.release == 'true'
         uses: mindsers/changelog-reader-action@32aa5b4c155d76c94e4ec883a223c947b2f02656 # v2.2.3
         with:
           path: charts/metrics-server/CHANGELOG.md
-          version: ${{ steps.chart_version.outputs.result }}
+          version: ${{ steps.setup.outputs.chart_version }}
 
       - name: Create release notes
-        if: steps.check_can_release.outputs.continue == 'true'
+        if: steps.setup.outputs.release == 'true'
         run: |
           set -euo pipefail
           cat <<"EOF" > charts/metrics-server/RELEASE.md
           ${{ steps.changelog_reader.outputs.changes }}
           EOF
 
       - name: Run chart-releaser
-        if: steps.check_can_release.outputs.continue == 'true'
+        if: steps.setup.outputs.release == 'true'
         uses: helm/chart-releaser-action@cae68fefc6b5f367a0275617c9f83181ba54714f # v1.7.0
         env:
           CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
diff --git a/charts/metrics-server/CHANGELOG.md b/charts/metrics-server/CHANGELOG.md
@@ -14,7 +14,7 @@
 
 ## [UNRELEASED]
 
-## [3.13.0] - TBC
+## [3.13.0] - 2025-07-22
 
 ### Added
 
