Skip to content

Commit

Permalink
GitHub actions: include job name in artifact name.
Browse files Browse the repository at this point in the history
Otherwise a random job would "win" and the entire workflow only produces one artifact.
  • Loading branch information
octo committed Nov 23, 2023
1 parent ea10ace commit 3c3b58d
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,12 +17,10 @@ jobs:
build:
runs-on: ubuntu-latest
container: collectd/ci:${{ matrix.container_tag }}
continue-on-error: ${{ matrix.allow_failures }}
continue-on-error: false
strategy:
fail-fast: false
matrix:
# for tasks that are optional, use the continue-on-error option, to prevent a workflow from failing when the task fails
allow_failures: [ false ]
container_tag:
# debian family
- bullseye_amd64
Expand Down Expand Up @@ -63,7 +61,7 @@ jobs:
- name: Dump test logs
uses: actions/upload-artifact@v3
with:
name: Test suite log
name: ${{ matrix.container_tag }} test log
path: test-suite.log
- name: Install bzip2
run: |
Expand Down Expand Up @@ -114,7 +112,7 @@ jobs:
- name: Dump test logs
uses: actions/upload-artifact@v3
with:
name: Test suite log
name: ${{ matrix.container_tag }} test log
path: test-suite.log
- name: Install bzip2
run: |
Expand Down

0 comments on commit 3c3b58d

Please sign in to comment.