You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by dp...@apache.org on 2024/02/28 14:10:12 UTC
(superset) branch master updated: fix(ci): docker builds don't work from remote forks (#27282)
This is an automated email from the ASF dual-hosted git repository.
dpgaspar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git
The following commit(s) were added to refs/heads/master by this push:
new 0383bedaba fix(ci): docker builds don't work from remote forks (#27282)
0383bedaba is described below
commit 0383bedabaaf9724c4ad5e148196b7abd65c5180
Author: Maxime Beauchemin <ma...@gmail.com>
AuthorDate: Wed Feb 28 06:10:06 2024 -0800
fix(ci): docker builds don't work from remote forks (#27282)
---
.github/workflows/docker.yml | 20 ++++++++++++++------
1 file changed, 14 insertions(+), 6 deletions(-)
diff --git a/.github/workflows/docker.yml b/.github/workflows/docker.yml
index 819a2a8b04..f1d8581bd5 100644
--- a/.github/workflows/docker.yml
+++ b/.github/workflows/docker.yml
@@ -51,9 +51,17 @@ jobs:
DOCKERHUB_TOKEN: ${{ secrets.DOCKERHUB_TOKEN }}
run: |
pip install click
- ./scripts/build_docker.py \
- ${{ matrix.build_preset }} \
- ${{ github.event_name }} \
- --build_context_ref "$RELEASE" $FORCE_LATEST \
- --platform "linux/arm64" \
- --platform "linux/amd64"
+ if [ "${{ github.event_name }}" = "push" ]; then
+ ./scripts/build_docker.py \
+ ${{ matrix.build_preset }} \
+ ${{ github.event_name }} \
+ --build_context_ref "$RELEASE" $FORCE_LATEST \
+ --platform "linux/arm64" \
+ --platform "linux/amd64"
+ elif [ "${{ github.event_name }}" = "pull_request" ]; then
+ ./scripts/build_docker.py \
+ ${{ matrix.build_preset }} \
+ ${{ github.event_name }} \
+ --build_context_ref "$RELEASE" $FORCE_LATEST \
+ --platform "linux/amd64"
+ fi