fix container permissions

neuralmagic · May 8, 2024 · 9333f65 · 9333f65
1 parent 4ac0867
commit 9333f65
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/.github/workflows/build-container.yml b/.github/workflows/build-container.yml
@@ -17,6 +17,9 @@ on:
 jobs:
   build-container:
     runs-on: ${{ inputs.build-label }}
+    permissions:
+      contents: read
+      packages: write
     steps:
       - name: Checkout code
         uses: actions/checkout@v3

diff --git a/.github/workflows/build-wheel-and-container.yml b/.github/workflows/build-wheel-and-container.yml
@@ -39,8 +39,8 @@ jobs:
       - name: Set variables for workflows
         id: set-outputs
         run: |
-          echo "dev=$DEV" >> $GITHUB_OUTPUT
-          echo "release=$RELEASE" >> $GITHUB_OUTPUT
+          echo "dev=false" >> $GITHUB_OUTPUT
+          echo "release=false" >> $GITHUB_OUTPUT
 
   build-wheel-and-push:
     needs: set-outputs
@@ -55,6 +55,7 @@ jobs:
       python: '3.10'
     secrets: inherit
 
+  """
   test-wheel-and-publish:
     needs: [set-outputs, build-wheel-and-push]
     uses: ./.github/workflows/test-wheel-and-publish.yml
@@ -65,10 +66,10 @@ jobs:
       dev: ${{ needs.set-outputs.outputs.dev }}
       release: ${{ needs.set-outputs.outputs.release  }}
     secrets: inherit
-
+  """
   # TODO: add release container build steps once 
   build-container-and-push:
-    needs: [set-outputs, test-wheel-and-publish]
+    needs: set-outputs
     uses: ./.github/workflows/build-container.yml
     with:
       build-label: aws-avx2-64G