-
Notifications
You must be signed in to change notification settings - Fork 450
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: Fix CI failure when building manylinux image (#4122)
## What do these changes do? - Return the error messages Immediately when executing the image building task through `gsctl` - Support treat `graphlearn` as an option to build `graphscope-client` wheel package Co-authored-by: xiaolei.zl <[email protected]>
- Loading branch information
1 parent
e69a05a
commit 83995e0
Showing
26 changed files
with
440 additions
and
628 deletions.
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,56 @@ | ||
name: Build GraphScope Wheel Images | ||
|
||
# build `graphscope/graphscope-dev:wheel-{v6d_version}` image based on manylinux, | ||
# including all dependencies for building graphscope wheel package. | ||
on: | ||
workflow_dispatch: | ||
inputs: | ||
v6d_version: | ||
description: 'Version for Vineyard (v6d)' | ||
required: true | ||
default: 'main' | ||
pull_request: | ||
branches: | ||
- main | ||
paths: | ||
- 'python/graphscope/gsctl/scripts/**' | ||
- '.github/workflows/build-graphscope-dev-wheel-images.yml' | ||
|
||
concurrency: | ||
group: ${{ github.repository }}-${{ github.event.number || github.head_ref || github.sha }}-${{ github.workflow }} | ||
cancel-in-progress: true | ||
|
||
env: | ||
REGISTRY: registry.cn-hongkong.aliyuncs.com | ||
|
||
jobs: | ||
build-wheel-image-amd64: | ||
runs-on: ubuntu-20.04 | ||
steps: | ||
- uses: actions/checkout@v4 | ||
with: | ||
submodules: true | ||
|
||
- name: Build Image | ||
run: | | ||
# build wheel image with specified v6d's version | ||
cd ${GITHUB_WORKSPACE}/k8s | ||
VINEYARD_VERSION=${{ github.event.inputs.v6d_version }} | ||
if [[ -n ${VINEYARD_VERSION} ]]; then | ||
# graphscope/graphscope-dev:wheel-<v6d_version>-amd64 | ||
make dev-wheel VINEYARD_VERSION=${VINEYARD_VERSION} | ||
else | ||
# pull_request: use default vineyard_version | ||
make dev-wheel | ||
fi | ||
- name: Release Image | ||
if: ${{ github.event_name == 'workflow_dispatch' }} | ||
env: | ||
docker_password: ${{ secrets.DOCKER_PASSWORD }} | ||
docker_username: ${{ secrets.DOCKER_USER }} | ||
run: | | ||
echo "${docker_password}" | sudo docker login --username="${docker_username}" ${{ env.REGISTRY }} --password-stdin | ||
VINEYARD_VERSION=${{ github.event.inputs.v6d_version }} | ||
sudo docker tag graphscope/graphscope-dev:wheel-${VINEYARD_VERSION}-amd64 ${{ env.REGISTRY }}/graphscope/graphscope-dev:wheel-${VINEYARD_VERSION}-amd64 | ||
sudo docker push ${{ env.REGISTRY }}/graphscope/graphscope-dev:wheel-${VINEYARD_VERSION}-amd64 |
106 changes: 0 additions & 106 deletions
106
.github/workflows/build-graphscope-manylinux-ext-images.yml
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,45 @@ | ||
name: Build GraphScope Manylinux Images | ||
|
||
# build `graphscope/manylinux2014` image based on centos7, including all necessary | ||
# dependencies except vineyard | ||
on: | ||
workflow_dispatch: | ||
pull_request: | ||
branches: | ||
- main | ||
paths: | ||
- 'python/graphscope/gsctl/scripts/**' | ||
- '.github/workflows/build-graphscope-manylinux-images.yml' | ||
|
||
|
||
concurrency: | ||
group: ${{ github.repository }}-${{ github.event.number || github.head_ref || github.sha }}-${{ github.workflow }} | ||
cancel-in-progress: true | ||
|
||
env: | ||
REGISTRY: registry.cn-hongkong.aliyuncs.com | ||
|
||
jobs: | ||
build-manylinux-image-amd64: | ||
runs-on: ubuntu-20.04 | ||
steps: | ||
- uses: actions/checkout@v4 | ||
with: | ||
submodules: true | ||
|
||
- name: Build Image | ||
run: | | ||
cd ${GITHUB_WORKSPACE}/k8s | ||
# output: graphscope/manylinux:amd64 | ||
make manylinux2014 | ||
- name: Release Image | ||
if: ${{ github.event_name == 'workflow_dispatch' }} | ||
env: | ||
docker_password: ${{ secrets.DOCKER_PASSWORD }} | ||
docker_username: ${{ secrets.DOCKER_USER }} | ||
run: | | ||
echo "${docker_password}" | sudo docker login --username="${docker_username}" ${{ env.REGISTRY }} --password-stdin | ||
# manylinux2014 image | ||
sudo docker tag graphscope/manylinux2014:amd64 ${{ env.REGISTRY }}/graphscope/manylinux2014:amd64 | ||
sudo docker push ${{ env.REGISTRY }}/graphscope/manylinux2014:amd64 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.