fix: try to address syntax issues (#19)

Signed-off-by: Aviral Takkar <avtakkar@microsoft.com>
Azure · Jul 24, 2024 · 20852f8 · 20852f8
1 parent a762ed4
commit 20852f8
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -58,7 +58,7 @@ jobs:
       contents: read
       id-token: write # This is required for requesting the JWT from AAD.
     env:
-      TAG: ${GITHUB_REF#refs/tags/}
+      TAG: ${{ needs.publish.outputs.get_git_tag.outputs.git_tag }}
     steps:
       - name: 'Az CLI login'
         uses: azure/login@v1
@@ -71,8 +71,9 @@ jobs:
         if: ${{ success() }}
         uses: actions/checkout@v2
         with:
-          ref: ${{ needs.publish.outputs.get_git_tag.outputs.git_tag }}
+          ref: ${{ env.TAG }}
 
       - name: 'Make'
+        if: ${{ success() }}
         run: |
-          PEERD_IMAGE_TAG=${{ needs.publish.outputs.get_git_tag.outputs.git_tag }} make tests-deps-install tests-ci-aks
+          PEERD_IMAGE_TAG=${{ env.TAG }} make tests-deps-install tests-ci-aks