Upload artifacts despite possible test failures

Reviewed-by: Neil Horman <nhorman@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/27089)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 6e81d8e086b656808f060b03b84bbc8575c37fb4..dcc258a7c1a52007f5ab2dea42b44df512e30441 100644 (file)
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -99,6 +99,7 @@ jobs:
       run: |
         util/wrap.pl -fips apps/openssl list -providers | grep 'name: CI FIPS Provider for OpenSSL$'
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@basic-gcc"
@@ -121,6 +122,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@basic-clang"
@@ -143,6 +145,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@linux-arm64"
@@ -184,6 +187,7 @@ jobs:
           ./util/opensslwrap.sh version -c
           .github/workflows/make-test
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@BSD-x86_64"
@@ -206,6 +210,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@minimal"
@@ -228,6 +233,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@no-deprecated"
@@ -250,6 +256,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@no-shared-ubuntu"
@@ -276,6 +283,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@no-shared-${{ matrix.os }}"
@@ -302,6 +310,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test OPENSSL_TEST_RAND_ORDER=0 TESTS="-test_fuzz* -test_ssl_* -test_sslapi -test_evp -test_cmp_http -test_verify -test_cms -test_store -test_enc -[01][0-9]"
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@non-caching"
@@ -328,6 +337,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test OPENSSL_TEST_RAND_ORDER=0
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@address_ub_sanitizer"
@@ -354,6 +364,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test OPENSSL_TEST_RAND_ORDER=0 TESTS="test_fuzz*"
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@fuzz_tests"
@@ -382,6 +393,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test OPENSSL_TEST_RAND_ORDER=0
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@memory_sanitizer"
@@ -408,6 +420,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test V=1 TESTS="test_lhash test_threads test_internal_provider test_provfetch test_provider test_pbe test_evp_kdf test_pkcs12 test_store test_evp test_quic*"
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@threads_sanitizer"
@@ -432,6 +445,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@enable_non-default_options"
@@ -462,6 +476,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@full_featured"
@@ -484,6 +499,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@no-legacy"
@@ -506,6 +522,7 @@ jobs:
     - name: make test
       run: .github/workflows/make-test
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@legacy"
@@ -548,6 +565,7 @@ jobs:
       run: ../source/.github/workflows/make-test
       working-directory: ./build
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@out-of-readonly-source-and-install-ubuntu"
@@ -592,6 +610,7 @@ jobs:
       run: ../source/.github/workflows/make-test
       working-directory: ./build
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "ci@out-of-readonly-source-and-install-${{ matrix.os }}"

diff --git a/.github/workflows/cross-compiles.yml b/.github/workflows/cross-compiles.yml
index fe2d4de308fd57ddf67f1594b4df2d5bebba03ed..bb5eaa5bea7940a382a90ccd58bfc7b1d8ecf16d 100644 (file)
--- a/.github/workflows/cross-compiles.yml
+++ b/.github/workflows/cross-compiles.yml
@@ -225,6 +225,7 @@ jobs:
                   TESTS="test_evp*" \
                   QEMU_LD_PREFIX=/usr/${{ matrix.platform.arch }}
     - name: save artifacts
+      if: success() || failure()
       uses: actions/upload-artifact@v4
       with:
         name: "cross-compiles@${{ matrix.platform.arch }}"