Merge pull request #4170 from jwcesign/ci

ci: fix uploading chart assests multiple times

.github/workflows/release.yml

@@ -40,8 +40,6 @@ jobs:
         files: |
           _output/release/${{ matrix.target }}-${{ matrix.os }}-${{ matrix.arch }}.tgz
           _output/release/${{ matrix.target }}-${{ matrix.os }}-${{ matrix.arch }}.tgz.sha256
-          _output/charts/karmada-chart-${{ github.ref_name }}.tgz
-          _output/charts/karmada-chart-${{ github.ref_name }}.tgz.sha256
   release-crds-assests:
     name: release crds
     runs-on: ubuntu-22.04
@@ -62,6 +60,22 @@ jobs:
       with:
         files: |
           crds.tar.gz
+  release-charts:
+    name: Release charts
+    runs-on: ubuntu-22.04
+    steps:
+    - uses: actions/checkout@v3
+    - name: Making helm charts
+      env:
+        VERSION: ${{ github.ref_name }}
+      run: make package-chart
+    - name: Uploading assets...
+      if: ${{ !env.ACT }}
+      uses: softprops/action-gh-release@v1
+      with:
+        files: |
+          _output/charts/karmada-chart-${{ github.ref_name }}.tgz
+          _output/charts/karmada-chart-${{ github.ref_name }}.tgz.sha256
   update-krew-index:
     needs: release-assests
     name: Update krew-index