[fix] Use pr-only mode for PRs only (#4686)

## Which problem is this PR solving? - There was a bug introduced in #4496 where `pr-only` mode was used unconditionally, instead of for PRs-only. ## Description of the changes - fork the build step into on-main and no-on-main Signed-off-by: Yuri Shkuro <[email protected]>
jaegertracing · Aug 18, 2023 · 17e2152 · 17e2152
1 parent 2119f5f
commit 17e2152
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/.github/workflows/ci-all-in-one-build.yml b/.github/workflows/ci-all-in-one-build.yml
@@ -43,7 +43,12 @@ jobs:
 
     - uses: docker/setup-qemu-action@2b82ce82d56a2a04d2637cd93a637ae1b359c0a7
 
+    - name: Build only linux/amd64 docker image for Pull Request
+      if: github.ref_name != 'main'
+      run: bash scripts/build-all-in-one-image.sh pr-only
+
     - name: Build, test, and publish all-in-one image
+      if: github.ref_name == 'main'
       run: bash scripts/build-all-in-one-image.sh pr-only
       env:
         DOCKERHUB_TOKEN: ${{ secrets.DOCKERHUB_TOKEN }}