Add new build-container workflow for PRs (#198)

* Add new build-container workflow for PRs

Signed-off-by: Jose Castillo Lema <[email protected]>

* Unify in one workflow

Signed-off-by: Jose Castillo Lema <[email protected]>

* Update description

Signed-off-by: Jose Castillo Lema <[email protected]>

* Removing the gha-build dependency from gha-push

Signed-off-by: Jose Castillo Lema <[email protected]>

* Differentiate job names

Signed-off-by: Jose Castillo Lema <[email protected]>

* Remove double quotes

Signed-off-by: Jose Castillo Lema <[email protected]>

* only run push in main branch

Signed-off-by: Jose Castillo Lema <[email protected]>

---------

Signed-off-by: Jose Castillo Lema <[email protected]>

Author: josecastillolema

.github/workflows/build-container.yml

@@ -1,7 +1,12 @@
-name: Release k8s-netperf container image
+name: Build k8s-netperf container image
 
 on:
+  pull_request:
+    branches: [ "*" ]
+    paths:
+      - "containers/**" # Trigger only when Dockerfile changes in a pull request
   push:
+    branches: [ main ]
     paths:
       - "containers/**" # Trigger only when Dockerfile changes in a pull request
 
@@ -18,7 +23,7 @@ jobs:
     permissions:
       contents: read
       packages: write
-    
+
     steps:
       - name: Setup QEMU & Install Dependecies
         run: |
@@ -35,13 +40,20 @@ jobs:
         with:
           go-version: ${{ env.GO_VER }}
 
-      - name: Build and Push multi-arch Image
-        id: push
+      - name: Build multi-arch Image
+        id: build
         run: |
           make build
-          podman login quay.io -u ${QUAY_USER} -p ${QUAY_TOKEN}
-          make gha-push
+          make gha-build
         env:
           GO_VER: ${{ env.GO_VER }}
+
+      - name: Push multi-arch Image
+        id: push
+        if: github.event_name == 'push'
+        run: |
+          podman login ${{ env.CONTAINER_REGISTRY }} -u ${QUAY_USER} -p ${QUAY_TOKEN}
+          make gha-push
+        env:
           QUAY_USER: ${{ secrets.QUAY_USER }}
-          QUAY_TOKEN: ${{ secrets.QUAY_TOKEN }} 
+          QUAY_TOKEN: ${{ secrets.QUAY_TOKEN }}

Makefile

@@ -46,7 +46,7 @@ gha-build:
 		--build-arg RHEL_VERSION=$(RHEL_VERSION) --platform=linux/amd64,linux/arm64,linux/ppc64le,linux/s390x \
 		./containers --manifest=$(CONTAINER_NS)/${BIN}:latest
 
-gha-push: gha-build
+gha-push:
 	@echo "Pushing Container Images & manifest"
 	$(CONTAINER) manifest push $(CONTAINER_NS)/${BIN}:latest $(CONTAINER_NS)/${BIN}:latest