Merge pull request #6 from ninech/fix-chart-workflow-2

use only the tag name instead of the full ref
ninech · Dec 5, 2024 · ff93bd1 · ff93bd1
2 parents 8ce42a2 + 73760bb
commit ff93bd1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/build-and-publish-images.yml b/.github/workflows/build-and-publish-images.yml
@@ -10,10 +10,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
 
       - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v1
+        uses: docker/setup-buildx-action@v3
 
       - name: Login to GitHub Container Registry
         uses: docker/login-action@v3

diff --git a/.github/workflows/chart.yaml b/.github/workflows/chart.yaml
@@ -34,5 +34,5 @@ jobs:
       - name: Build, package and push helm chart
         run: |
           ./tool/go run cmd/k8s-operator/generate/main.go helmcrd
-          ./tool/helm package --app-version="${GITHUB_REF}" --version=${GITHUB_REF} './cmd/k8s-operator/deploy/chart'
-          ./tool/helm push ./tailscale-operator-${GITHUB_REF}.tgz oci://${{ steps.set-variables.outputs.REPOSITORY }}/charts
+          ./tool/helm package --app-version=${{  github.ref_name }} --version=${{  github.ref_name }} './cmd/k8s-operator/deploy/chart'
+          ./tool/helm push ./tailscale-operator-${{  github.ref_name }}.tgz oci://${{ steps.set-variables.outputs.REPOSITORY }}/charts