Merge pull request #536 from LedgerHQ/INFRAPRJ-7379_1

[ci][fix] INFRAPRJ-7379: Use proper push switcher in context of 'image' output
LedgerHQ · Dec 6, 2024 · d65cd29 · d65cd29
2 parents 7c39576 + 282945e
commit d65cd29
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/.github/workflows/speculos-builder.yml b/.github/workflows/speculos-builder.yml
@@ -37,6 +37,13 @@ jobs:
           platform=${{ matrix.platform }}
           echo "PLATFORM_PAIR=${platform//\//-}" >> $GITHUB_ENV
 
+          if [[ "${{ github.event_name }}" == 'push' && "${{ github.ref }}" == 'refs/heads/master' ]]; then
+             PUSH_FLAG='true'
+          else
+             PUSH_FLAG='false'
+          fi
+          echo "PUSH_FLAG=${PUSH_FLAG}" >> $GITHUB_ENV
+
       - name: Clone
         uses: actions/checkout@v4
 
@@ -57,10 +64,9 @@ jobs:
         uses: docker/build-push-action@v6
         id: build
         with:
-          push: ${{ github.event_name == 'push' && github.ref == 'refs/heads/master' }}
           file: build.Dockerfile
           platforms: ${{ matrix.platform }}
-          outputs: type=image,name=ghcr.io/ledgerhq/speculos-builder,push-by-digest=true,name-canonical=true,push=true
+          outputs: type=image,name=ghcr.io/ledgerhq/speculos-builder,push-by-digest=true,name-canonical=true,push=${{ env.PUSH_FLAG }}
 
       - name: Export digest
         run: |