Skip to content

Commit

Permalink
[improve][ci] Upgrade to actions/upload-artifact@v4 (#21836)
Browse files Browse the repository at this point in the history
  • Loading branch information
lhotari authored Jan 2, 2024
1 parent cf6b5d0 commit 0ba5ae6
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 15 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/pulsar-ci-flaky.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,7 @@ jobs:
- name: Upload Jacoco report files to build artifacts
if: ${{ needs.preconditions.outputs.collect_coverage == 'true' }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: Jacoco-coverage-report-flaky
path: target/jacoco_test_coverage_report_flaky.zip
Expand All @@ -246,7 +246,7 @@ jobs:
flags: unittests

- name: Upload Surefire reports
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ !success() || env.TRACE_TEST_RESOURCE_CLEANUP != 'off' }}
with:
name: Unit-BROKER_FLAKY-surefire-reports
Expand All @@ -255,7 +255,7 @@ jobs:
if-no-files-found: ignore

- name: Upload possible heap dump, core dump or crash files
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ always() }}
with:
name: Unit-BROKER_FLAKY-dumps
Expand Down
24 changes: 12 additions & 12 deletions .github/workflows/pulsar-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -345,15 +345,15 @@ jobs:
fi
- name: Upload Surefire reports
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ !success() || env.TRACE_TEST_RESOURCE_CLEANUP != 'off' }}
with:
name: Unit-${{ matrix.group }}-surefire-reports
path: surefire-reports
retention-days: 7

- name: Upload possible heap dump, core dump or crash files
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ always() }}
with:
name: Unit-${{ matrix.group }}-dumps
Expand Down Expand Up @@ -433,7 +433,7 @@ jobs:
zip -qr jacoco_test_coverage_report_unittests.zip jacoco_test_coverage_report || true
- name: Upload Jacoco report files to build artifacts
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: Jacoco-coverage-report-unittests
path: target/jacoco_test_coverage_report_unittests.zip
Expand Down Expand Up @@ -673,15 +673,15 @@ jobs:
annotate_only: 'true'

- name: Upload Surefire reports
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ !success() }}
with:
name: Integration-${{ matrix.group }}-surefire-reports
path: surefire-reports
retention-days: 7

- name: Upload container logs
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ !success() }}
continue-on-error: true
with:
Expand Down Expand Up @@ -760,7 +760,7 @@ jobs:
zip -qr jacoco_test_coverage_report_inttests.zip jacoco_test_coverage_report jacoco_inttest_coverage_report || true
- name: Upload Jacoco report files to build artifacts
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: Jacoco-coverage-report-inttests
path: target/jacoco_test_coverage_report_inttests.zip
Expand Down Expand Up @@ -1036,7 +1036,7 @@ jobs:
annotate_only: 'true'

- name: Upload container logs
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ !success() }}
continue-on-error: true
with:
Expand All @@ -1045,7 +1045,7 @@ jobs:
retention-days: 7

- name: Upload Surefire reports
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ !success() }}
with:
name: System-${{ matrix.name }}-surefire-reports
Expand Down Expand Up @@ -1124,7 +1124,7 @@ jobs:
zip -qr jacoco_test_coverage_report_systests.zip jacoco_test_coverage_report jacoco_inttest_coverage_report || true
- name: Upload Jacoco report files to build artifacts
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: Jacoco-coverage-report-systests
path: target/jacoco_test_coverage_report_systests.zip
Expand Down Expand Up @@ -1243,7 +1243,7 @@ jobs:
annotate_only: 'true'

- name: Upload container logs
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ !success() }}
continue-on-error: true
with:
Expand All @@ -1252,7 +1252,7 @@ jobs:
retention-days: 7

- name: Upload Surefire reports
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ !success() }}
with:
name: System-${{ matrix.name }}-surefire-reports
Expand Down Expand Up @@ -1406,7 +1406,7 @@ jobs:
-pl '!distribution/server,!distribution/io,!distribution/offloaders,!tiered-storage/file-system,!pulsar-io/flume,!pulsar-io/hbase,!pulsar-io/hdfs2,!pulsar-io/hdfs3,!pulsar-io/docs,!pulsar-io/jdbc/openmldb'
- name: Upload report
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: ${{ cancelled() || failure() }}
continue-on-error: true
with:
Expand Down

0 comments on commit 0ba5ae6

Please sign in to comment.