revert PR builds to not publish, add correct user to main build

Signed-off-by: Jade Carino <carino_jade@yahoo.co.uk>
galasa-dev · Sep 30, 2024 · 5bf9c2f · 5bf9c2f
1 parent 3c29bd6
commit 5bf9c2f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 11 deletions.
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -142,8 +142,8 @@ jobs:
         uses: docker/login-action@v3
         with:
           registry: ${{ env.REGISTRY }}
-          username: ${{ github.actor }}
-          password: ${{ secrets.GITHUB_TOKEN }}
+          username: galasa-team
+          password: ${{ secrets.GALASA_TEAM_WRITE_PACKAGES_TOKEN }}
 
       - name: Extract metadata for Wrapping image
         id: metadata

diff --git a/.github/workflows/pr-build.yaml b/.github/workflows/pr-build.yaml
@@ -85,13 +85,6 @@ jobs:
           -Dgalasa.release.repo=file:${{ github.workspace }}/repo \
           --batch-mode --errors --fail-at-end \
           --settings /home/runner/work/gpg/settings.xml 
-
-      - name: Login to Github Container Registry
-        uses: docker/login-action@v3
-        with:
-          registry: ${{ env.REGISTRY }}
-          username: galasa-team
-          password: ${{ secrets.GALASA_TEAM_WRITE_PACKAGES_TOKEN }}
       
       - name: Extract metadata for Wrapping image
         id: metadata
@@ -106,8 +99,7 @@ jobs:
         with:
           context: .
           file: dockerfiles/dockerfile.wrapping
-          # push: false
-          push: true
+          push: false
           tags: ${{ steps.metadata.outputs.tags }}
           labels: ${{ steps.metadata.outputs.labels }}
           build-args: |