ci(test): skip test artifact upload on success

It's only needed on failure, we can save time otherwise. (cherry picked from commit 0ca476e)
camunda · Apr 21, 2023 · f0a1f8b · f0a1f8b
1 parent ac937b7
commit f0a1f8b
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -106,7 +106,7 @@ jobs:
           buildOutputFilePath: ${{ env.BUILD_OUTPUT_FILE_PATH }}
       - name: Upload test artifacts
         uses: ./.github/actions/collect-test-artifacts
-        if: always()
+        if: failure()
         with:
           name: "[IT] ${{ matrix.name }}"
   unit-tests:
@@ -146,7 +146,7 @@ jobs:
           buildOutputFilePath: ${{ env.BUILD_OUTPUT_FILE_PATH }}
       - name: Upload test artifacts
         uses: ./.github/actions/collect-test-artifacts
-        if: always()
+        if: failure()
         with:
           name: "unit tests"
   smoke-tests:
@@ -204,7 +204,7 @@ jobs:
           verify
       - name: Upload test artifacts
         uses: ./.github/actions/collect-test-artifacts
-        if: always()
+        if: failure()
         with:
           name: "[Smoke] ${{ matrix.os }} with ${{ matrix.arch }}"
   property-tests:
@@ -240,7 +240,7 @@ jobs:
           buildOutputFilePath: ${{ env.BUILD_OUTPUT_FILE_PATH }}
       - name: Upload test artifacts
         uses: ./.github/actions/collect-test-artifacts
-        if: always()
+        if: failure()
         with:
           name: Property Tests
   go-client:
@@ -319,7 +319,7 @@ jobs:
           buildOutputFilePath: ${{ env.BUILD_OUTPUT_FILE_PATH }}
       - name: Upload test artifacts
         uses: ./.github/actions/collect-test-artifacts
-        if: always()
+        if: failure()
         with:
           name: Java 8 Client
   codeql: