Skip to content

Commit

Permalink
CI: Upgrade from upload-artifact v3 to v4 (#1309)
Browse files Browse the repository at this point in the history
This should make the test log artifacts available for download
immediately, allowing the test-matrix to include the most recent test
results.

Details:

https://github.blog/changelog/2023-12-14-github-actions-artifacts-v4-is-now-generally-available/
  • Loading branch information
korran authored Feb 2, 2024
1 parent 2b7ff69 commit 472276c
Show file tree
Hide file tree
Showing 7 changed files with 21 additions and 21 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/fpga-image.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
sudo bash build.sh
- name: 'Upload image as artifact'
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: caliptra-fpga-image
path: ci-tools/fpga-image/out/image.img
Expand Down
24 changes: 12 additions & 12 deletions .github/workflows/fpga.yml
Original file line number Diff line number Diff line change
Expand Up @@ -76,15 +76,15 @@ jobs:

- name: 'Upload FPGA bitstream artifact'
if: steps.restore_rtl_cache.outputs.cache-hit
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: caliptra-fpga-bitstream${{ inputs.artifact-suffix }}
path: /tmp/caliptra-fpga-bitstream/caliptra_fpga.bin
retention-days: 7

- name: 'Upload kernel module artifacts'
if: steps.restore_kmod_cache.outputs.cache-hit
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: caliptra-fpga-kmod${{ inputs.artifact-suffix }}
path: /tmp/caliptra-fpga-kmod/
Expand Down Expand Up @@ -171,7 +171,7 @@ jobs:
mksquashfs /tmp/caliptra-test-binaries /tmp/caliptra-test-binaries.sqsh -comp zstd
- name: 'Upload test binaries artifact'
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: caliptra-test-binaries${{ inputs.artifact-suffix }}
path: /tmp/caliptra-test-binaries.sqsh
Expand All @@ -183,7 +183,7 @@ jobs:
cargo run --release -p caliptra-builder -- --all_elfs /tmp/caliptra-test-firmware
- name: 'Upload test firmware artifact'
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: caliptra-test-firmware${{ inputs.artifact-suffix }}
path: /tmp/caliptra-test-firmware
Expand Down Expand Up @@ -246,7 +246,7 @@ jobs:
key: ${{ needs.check_cache.outputs.kmod_cache_key }}

- name: 'Upload kernel module artifacts'
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: caliptra-fpga-kmod${{ inputs.artifact-suffix }}
path: /tmp/caliptra-fpga-kmod/
Expand Down Expand Up @@ -286,7 +286,7 @@ jobs:
fi
- name: 'Upload FPGA bitstream artifact'
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: caliptra-fpga-bitstream${{ inputs.artifact-suffix }}
path: hw-latest/fpga/caliptra_build/caliptra_fpga.bin
Expand All @@ -302,7 +302,7 @@ jobs:

steps:
- name: 'Download FPGA Bitstream Artifact'
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: caliptra-fpga-bitstream${{ inputs.artifact-suffix }}
path: /tmp/caliptra-fpga-bitstream
Expand Down Expand Up @@ -332,25 +332,25 @@ jobs:
git submodule update --init dpe
- name: 'Download FPGA Bitstream Artifact'
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: caliptra-fpga-bitstream${{ inputs.artifact-suffix }}
path: /tmp/caliptra-fpga-bitstream

- name: 'Download kernel driver artifacts'
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: caliptra-fpga-kmod${{ inputs.artifact-suffix }}
path: /tmp/caliptra-fpga-kmod/

- name: 'Download Test Binaries Artifact'
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: caliptra-test-binaries${{ inputs.artifact-suffix }}
path: /tmp/caliptra-test-binaries.sqsh

- name: 'Download Test Firmware Artifact'
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: caliptra-test-firmware${{ inputs.artifact-suffix }}
path: /tmp/caliptra-test-firmware
Expand Down Expand Up @@ -433,7 +433,7 @@ jobs:
--profile=nightly
- name: 'Upload test results'
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: success() || failure()
with:
name: caliptra-test-results${{ inputs.artifact-suffix }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/fw-test-emu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ jobs:
--profile=nightly
- name: 'Upload test results'
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: success() || failure()
with:
name: caliptra-test-results${{ inputs.artifact-suffix }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/nightly-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@ jobs:
mv ./release/release.zip ./release/caliptra_${{ needs.find-latest-release.outputs.new_release_tag }}.zip
- name: 'Download all artifacts'
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
path: /tmp/artifacts

Expand Down Expand Up @@ -209,7 +209,7 @@ jobs:
echo "caliptra_${{ needs.find-latest-release.outputs.new_release_tag }}.zip" > /tmp/release-info/zip-file-name
- name: Write artifact with release info
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: release-info
path: /tmp/release-info
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/policy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:
- name: Check that the ROM hash matches the frozen one
run: ./ci.sh check_frozen_images

- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
if: always()
with:
name: caliptra-rom.elf
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/reusable-aflplusplus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -152,13 +152,13 @@ jobs:
key: ${{ inputs.name }}-${{ env.CACHE_BUSTER }}

- name: Archive test cases dir
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: test_cases
path: ${{ inputs.fuzz_target_path }}/test_cases/

- name: Archive fuzzing coverage
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: plot
path: ${{ inputs.fuzz_target_path }}/plot/
4 changes: 2 additions & 2 deletions .github/workflows/reusable-libfuzzer.yml
Original file line number Diff line number Diff line change
Expand Up @@ -115,13 +115,13 @@ jobs:
key: ${{ inputs.name }}-${{ env.CACHE_BUSTER }}

- name: Archive test cases dir
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: test_cases
path: ${{ inputs.fuzz_target_path }}/test_cases/

- name: Archive fuzzing coverage
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: coverage
path: ${{ inputs.fuzz_target_path }}/index.html

0 comments on commit 472276c

Please sign in to comment.