Skip to content

Commit

Permalink
ci: Fixes release failure of groot image (#3337)
Browse files Browse the repository at this point in the history
- Always clean the Groot image after building
- Simplify the release image manifest creation and pushing from the version

## Related issue number

<!-- Are there any issues opened that will be resolved by merging this
change? -->

Fixes
  • Loading branch information
lidongze0629 committed Nov 8, 2023
1 parent aac5637 commit 8235b29
Showing 1 changed file with 2 additions and 10 deletions.
12 changes: 2 additions & 10 deletions .github/workflows/release.yml
Expand Up @@ -139,14 +139,15 @@ jobs:
sudo docker rmi -f ${{ env.GSS_IMAGE }}:${tag} || true
- name: Clean Release Image
if: ${{ github.event_name == 'push' && startsWith(github.ref, 'refs/tags/v') && github.repository == 'alibaba/GraphScope' }}
if: always()
run: |
# aarch64
arch=$(uname -m)
sudo docker rmi -f graphscope-store:${SHORT_SHA} || true
sudo docker rmi -f ${{ env.GSS_IMAGE }}:${{ steps.tag.outputs.TAG }}-${arch} || true
push-gss-image-manifest:
if: ${{ github.event_name == 'push' && startsWith(github.ref, 'refs/tags/v') && github.repository == 'alibaba/GraphScope' }}
runs-on: ubuntu-20.04

needs: [build-gss-image-x86-64, build-gss-image-aarch64]
Expand All @@ -163,22 +164,13 @@ jobs:
run: |
echo "${docker_password}" | sudo docker login --username="${docker_username}" ${{ env.DOCKER_URL }} --password-stdin
time=$(date "+%Y%m%d")
version=$(cat ${GITHUB_WORKSPACE}/VERSION)
# create
sudo docker manifest create \
${{ env.GSS_IMAGE }}:${version}a${time} \
${{ env.GSS_IMAGE }}:${version}a${time}-x86_64 \
${{ env.GSS_IMAGE }}:${version}a${time}-aarch64
sudo docker manifest create \
${{ env.GSS_IMAGE }}:${{ steps.tag.outputs.TAG }} \
${{ env.GSS_IMAGE }}:${{ steps.tag.outputs.TAG }}-x86_64 \
${{ env.GSS_IMAGE }}:${{ steps.tag.outputs.TAG }}-aarch64
# push
sudo docker manifest push ${{ env.GSS_IMAGE }}:${version}a${time}
sudo docker manifest push ${{ env.GSS_IMAGE }}:${{ steps.tag.outputs.TAG }}
release-helm-charts:
Expand Down

0 comments on commit 8235b29

Please sign in to comment.