CI: fix chart release flow, skip the exist chart version. (#820)

Signed-off-by: wuhuizuo <wuhuizuo@126.com> --------- Signed-off-by: wuhuizuo <wuhuizuo@126.com>
PingCAP-QE · Dec 4, 2023 · 469ac5d · 469ac5d
1 parent 1817fa0
commit 469ac5d
Show file tree

Hide file tree

Showing 7 changed files with 56 additions and 51 deletions.
diff --git a/.github/workflows/charts_lint-test.yaml b/.github/workflows/charts_lint-test.yaml
@@ -0,0 +1,43 @@
+name: Charts Lint Test
+
+on:
+  pull_request:
+    branches: [main]
+    paths:
+      - "charts/**"
+
+jobs:
+  chart-test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Set up Helm
+        uses: azure/setup-helm@v3
+        with:
+          version: v3.8.1
+
+      - uses: actions/setup-python@v4
+        with:
+          python-version: 3.7
+
+      - name: Set up chart-testing
+        uses: helm/chart-testing-action@v2.6.1
+
+      - name: Run chart-testing (list-changed)
+        id: list-changed
+        run: |
+          changed=$(ct list-changed --target-branch ${{ github.event.repository.default_branch }})
+          if [[ -n "$changed" ]]; then
+            echo "::set-output name=changed::true"
+          fi
+
+      - name: Run chart-testing (lint)
+        run: |
+          # from sub `charts` dir to uplevel dir.
+          for d in `find . -name charts -type d | sort -r`; do
+            ct lint --all --debug --chart-dirs=${d#./}
+          done
diff --git a/.github/workflows/charts_release.yaml b/.github/workflows/charts_release.yaml
@@ -1,6 +1,7 @@
 name: Release Charts
 
 on:
+  workflow_dispatch:
   push:
     branches:
       - main
@@ -32,5 +33,7 @@ jobs:
 
       - name: Run chart-releaser
         uses: helm/chart-releaser-action@v1.6.0
+        with:
+          skip_existing: true
         env:
           CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
diff --git a/.github/workflows/charts_test-bazel-remote.yaml b/.github/workflows/charts_test-bazel-remote.yaml
@@ -5,7 +5,7 @@ on:
     branches: [main]
     paths:
       - "charts/bazel-remote/**"
-      - ".github/workflows/chart*"
+
 jobs:
   chart-test:
     runs-on: ubuntu-latest
@@ -27,21 +27,6 @@ jobs:
       - name: Set up chart-testing
         uses: helm/chart-testing-action@v2.6.1
 
-      - name: Run chart-testing (list-changed)
-        id: list-changed
-        run: |
-          changed=$(ct list-changed --target-branch ${{ github.event.repository.default_branch }})
-          if [[ -n "$changed" ]]; then
-            echo "::set-output name=changed::true"
-          fi
-
-      - name: Run chart-testing (lint)
-        run: |
-          # from sub `charts` dir to uplevel dir.
-          for d in `find . -name charts -type d | sort -r`; do
-            ct lint --all --debug --chart-dirs=${d#./}
-          done
-
       - name: Setup Kubernetes
         uses: engineerd/setup-kind@v0.5.0
         with:

diff --git a/.github/workflows/charts_test-jenkins.yaml b/.github/workflows/charts_test-jenkins.yaml
@@ -5,7 +5,7 @@ on:
     branches: [main]
     paths:
       - "charts/jenkins/**"
-      - ".github/workflows/chart*"
+
 jobs:
   chart-test:
     runs-on: ubuntu-latest
@@ -27,21 +27,6 @@ jobs:
       - name: Set up chart-testing
         uses: helm/chart-testing-action@v2.6.1
 
-      - name: Run chart-testing (list-changed)
-        id: list-changed
-        run: |
-          changed=$(ct list-changed --target-branch ${{ github.event.repository.default_branch }})
-          if [[ -n "$changed" ]]; then
-            echo "::set-output name=changed::true"
-          fi
-
-      - name: Run chart-testing (lint)
-        run: |
-          # from sub `charts` dir to uplevel dir.
-          for d in `find . -name charts -type d | sort -r`; do
-            ct lint --all --debug --chart-dirs=${d#./}
-          done
-
       - name: Setup Kubernetes
         uses: engineerd/setup-kind@v0.5.0
         with:

diff --git a/.github/workflows/charts_test-prow.yaml b/.github/workflows/charts_test-prow.yaml
@@ -5,7 +5,7 @@ on:
     branches: [main]
     paths:
       - "charts/prow/**"
-      - ".github/workflows/chart*"
+
 jobs:
   chart-test:
     runs-on: ubuntu-latest
@@ -27,21 +27,6 @@ jobs:
       - name: Set up chart-testing
         uses: helm/chart-testing-action@v2.6.1
 
-      - name: Run chart-testing (list-changed)
-        id: list-changed
-        run: |
-          changed=$(ct list-changed --target-branch ${{ github.event.repository.default_branch }})
-          if [[ -n "$changed" ]]; then
-            echo "::set-output name=changed::true"
-          fi
-
-      - name: Run chart-testing (lint)
-        run: |
-          # from sub `charts` dir to uplevel dir.
-          for d in `find . -name charts -type d | sort -r`; do
-            ct lint --all --debug --chart-dirs=${d#./}
-          done
-
       - name: Setup Kubernetes
         uses: engineerd/setup-kind@v0.5.0
         with:

diff --git a/charts/git-cdn/Chart.yaml b/charts/git-cdn/Chart.yaml
@@ -1,9 +1,12 @@
 apiVersion: v2
 name: git-cdn
-description: | 
+description: |
   It's a git mirror that you can place near your CI workers and developers.
-  It acts as a git+http(s) proxy, reducing WAN usage and accelerating git 
+  It acts as a git+http(s) proxy, reducing WAN usage and accelerating git
   access for large repositories.
+maintainers:
+  - name: wuhuizuo
+    email: wuhuizuo@126.com
 
 # A chart can be either an 'application' or a 'library' chart.
 #

diff --git a/charts/git-cdn/values.yaml b/charts/git-cdn/values.yaml
@@ -106,4 +106,5 @@ affinity: {}
 configuration:
   GITSERVER_UPSTREAM: https://github.com
   PACK_CACHE_SIZE_GB: 20
-  WORKING_DIRECTORY: /data # directory where to put cache files
+  # directory where to put cache files
+  WORKING_DIRECTORY: /data