Skip to content

Commit c116698

Browse files
committed
[SPARK-40746][INFRA] Fix Dockerfile build workflow
### What changes were proposed in this pull request? This patch is to make the workflow work in apache repo: - Add `.github/workflows/build_3.3.0.yaml` and `3.3.0/**` to trigger paths - Change `apache/spark-docker:TAG` to `ghcr.io/apache/spark-docker/spark:TAG` - Remove the push, we only need to build locally to validate dockerfile, even in future K8s IT test we can also refactor to use minikube docker, it still can be local build. ### Why are the changes needed? To make the workflow works well in apache repo. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? CI passed Closes: #5 Closes #7 from Yikun/SPARK-40746. Authored-by: Yikun Jiang <[email protected]> Signed-off-by: Yikun Jiang <[email protected]>
1 parent 30fd82f commit c116698

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

.github/workflows/build_3.3.0.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,8 @@ on:
2424
branches:
2525
- 'master'
2626
paths:
27-
- '3.3.0/'
27+
- '3.3.0/**'
28+
- '.github/workflows/build_3.3.0.yaml'
2829
- '.github/workflows/main.yml'
2930

3031
jobs:

.github/workflows/main.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -97,8 +97,7 @@ jobs:
9797
uses: docker/build-push-action@v2
9898
with:
9999
context: ${{ env.IMAGE_PATH }}
100-
push: true
101-
tags: ${{ env.TEST_REPO }}:${{ env.UNIQUE_IMAGE_TAG }}
100+
tags: ${{ env.TEST_REPO }}/${{ env.IMAGE_NAME }}:${{ env.UNIQUE_IMAGE_TAG }}
102101
platforms: linux/amd64,linux/arm64
103102

104103
- name: Image digest

0 commit comments

Comments
 (0)