Skip to content

Commit

Permalink
Fix Docker workflow after update to upload-artifact@v4 (#3757)
Browse files Browse the repository at this point in the history
* Fix Docker workflow after update to upload-artifact@v4

* Fix Artifact name for Docker job

---------

Co-authored-by: Luca Joss <[email protected]>
  • Loading branch information
romac and ljoss17 authored Jan 9, 2024
1 parent 3ec02fe commit 2c254ac
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions .github/workflows/docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,10 @@ jobs:
fail-fast: false
matrix:
platform:
- linux/amd64
- linux/arm64
- id: linux/amd64
name: amd64
- id: linux/arm64
name: arm64
steps:
- name: Checkout
uses: actions/checkout@v4
Expand Down Expand Up @@ -49,7 +51,7 @@ jobs:
with:
context: .
file: ./ci/release/hermes.Dockerfile
platforms: ${{ matrix.platform }}
platforms: ${{ matrix.platform.id }}
labels: ${{ steps.meta.outputs.labels }}
outputs: type=image,name=${{ env.REGISTRY_IMAGE }},push-by-digest=true,name-canonical=true,push=true
cache-from: type=gha
Expand All @@ -64,7 +66,7 @@ jobs:
- name: Upload digest
uses: actions/upload-artifact@v4
with:
name: digests
name: digests-${{ matrix.platform.name }}
path: /tmp/digests/*
if-no-files-found: error
retention-days: 1
Expand All @@ -77,7 +79,8 @@ jobs:
- name: Download digests
uses: actions/download-artifact@v4
with:
name: digests
pattern: digests-*
merge-multiple: true
path: /tmp/digests

- name: Set up Docker Buildx
Expand Down

0 comments on commit 2c254ac

Please sign in to comment.