From 47cd3f0c8df1d2513421e92e8c81a35a0a09c30a Mon Sep 17 00:00:00 2001
From: Dat WMDE <dat.nguyen@wikimedia.de>
Date: Thu, 22 Aug 2024 16:56:41 +0200
Subject: [PATCH] Run docker build on PR (#852)

* feat: run docker build on PR

* fix: remove 'main' under pull_request

* fix: build the image on Pull Requests

* fix: remove 'pull_request'

* fix: add 'pull_request'

* remove typo of push defined twice

* fix: add condition to run a few steps

* fix: add condition to update argo step

---------

Co-authored-by: Thomas Arrow <tarrow@users.noreply.github.com>
---
 .github/workflows/docker.build.yml | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/docker.build.yml b/.github/workflows/docker.build.yml
index 7464352a..a9c4cdf0 100644
--- a/.github/workflows/docker.build.yml
+++ b/.github/workflows/docker.build.yml
@@ -4,6 +4,8 @@ on:
   push:
     branches:
     - main
+  pull_request:
+
 jobs:
   docker-build:
     runs-on: ubuntu-latest
@@ -44,20 +46,23 @@ jobs:
           platforms: linux/amd64
           cache-from: type=local,src=/tmp/.buildx-cache
           cache-to: type=local,mode=max,dest=/tmp/.buildx-cache
+          push: ${{ github.event_name != 'pull_request' }}
           tags: ${{ steps.docker_meta.outputs.tags }}
           labels: ${{ steps.docker_meta.outputs.labels }}
-          push: true
       - name: Check out `wmde/wbaas-deploy` repository in staging child directory
+        if: github.event_name != 'pull_request'
         uses: actions/checkout@v3.5.3
         with:
           repository: wmde/wbaas-deploy
           path: ./repos/wbaas-deploy-staging
       - name: Check out `wmde/wbaas-deploy` repository in production child directory
+        if: github.event_name != 'pull_request'
         uses: actions/checkout@v3.5.3
         with:
           repository: wmde/wbaas-deploy
           path: ./repos/wbaas-deploy-production
       - name: Update helmfile values for local, staging and production
+        if: github.event_name != 'pull_request'
         id: update-helmfile-values
         run: |
           TAG="$(echo ${{ steps.docker_meta.outputs.tags }} | cut -d':' -f2)"
@@ -67,6 +72,7 @@ jobs:
           sed -i "/image:/{n;s/tag:.*/tag: $TAG/;}" ./repos/wbaas-deploy-production/k8s/helmfile/env/production/ui.values.yaml.gotmpl
           # run the values script for local, prod and staging
       - name: Update argo values for local, staging and production
+        if: github.event_name != 'pull_request'
         id: update-argo-values
         run: |
           cd ./repos/wbaas-deploy-staging
@@ -78,12 +84,14 @@ jobs:
           cd ../wbaas-deploy-production
           ./bin/generate-values production ui
       - name: Truncate commit message
+        if: github.event_name != 'pull_request'
         id: truncate-commit-message
         run: |
           MSG=$(git log -1 --pretty=format:%s)
           echo "msg=$MSG" >> $GITHUB_OUTPUT
       - name: Create Staging+Local Pull Request
         uses: peter-evans/create-pull-request@v5
+        if: github.event_name != 'pull_request'
         with:
           path: ./repos/wbaas-deploy-staging
           commit-message: 'Staging+Local: Deploy new UI image ${{ steps.docker_meta.outputs.tags }}'
@@ -99,6 +107,7 @@ jobs:
             **Changes**: [${{ steps.truncate-commit-message.outputs.msg }}](https://github.com/wbstack/ui/commit/${{ github.sha }})
       - name: Create Production Pull Request
         uses: peter-evans/create-pull-request@v5
+        if: github.event_name != 'pull_request'
         with:
           path: ./repos/wbaas-deploy-production
           commit-message: 'Production: Deploy new UI image ${{ steps.docker_meta.outputs.tags }}'