This is an automated email from the ASF dual-hosted git repository.
ethanfeng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/celeborn.git
The following commit(s) were added to refs/heads/main by this push:
new 192213daf [CELEBORN-1900][FOLLOWUP] fixed wrong CI parameter
192213daf is described below
commit 192213dafb84b0db1553b2d6ac9f4f48fba6af9e
Author: Björn Boschman <[email protected]>
AuthorDate: Mon Mar 24 11:27:18 2025 +0800
[CELEBORN-1900][FOLLOWUP] fixed wrong CI parameter
### What changes were proposed in this pull request?
The referred step ID used to extract the version was wrong
### Why are the changes needed?
To fix docker build workflow failures.
### Does this PR introduce _any_ user-facing change?
no
### How was this patch tested?
Manually tests.
Closes #3160 from jesusch/CELEBORN-1900-docker-images-fix.
Authored-by: Björn Boschman <[email protected]>
Signed-off-by: mingji <[email protected]>
---
.github/workflows/docker-build.yml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/.github/workflows/docker-build.yml
b/.github/workflows/docker-build.yml
index 33d0c75d6..cbdd25acd 100644
--- a/.github/workflows/docker-build.yml
+++ b/.github/workflows/docker-build.yml
@@ -28,13 +28,13 @@ jobs:
- name: Download Celeborn Release
run: |
- VERSION="${{ steps.extract_version.outputs.VERSION }}"
+ VERSION="${{ steps.set_version.outputs.VERSION }}"
wget
https://downloads.apache.org/celeborn/celeborn-${VERSION}/apache-celeborn-${VERSION}-bin.tgz
tar -xzf apache-celeborn-${VERSION}-bin.tgz
- name: Build and Push
run: |
- VERSION="${{ steps.extract_version.outputs.VERSION }}"
+ VERSION="${{ steps.set_version.outputs.VERSION }}"
cd apache-celeborn-${VERSION}-bin
docker login -u ${{ secrets.DOCKER_USERNAME }} -p ${{
secrets.DOCKER_PASSWORD }}
docker buildx create --use
