merge: #12506

12506: [Backport stable/8.0] ci(test): skip test artifact upload on success r=megglos a=backport-action

# Description
Backport of #12497 to `stable/8.0`.

relates to #12028

Co-authored-by: Meggle (Sebastian Bathke) <sebastian.bathke@camunda.com>

.github/workflows/ci.yml

@@ -38,7 +38,7 @@ jobs:
             name: "QA Integration Tests"
             maven-modules: "qa/integration-tests"
             maven-build-threads: 1
-            maven-test-fork-count: 8
+            maven-test-fork-count: 6
             tcc-enabled: 'true'
             tcc-concurrency: 3
           - group: qa-update
@@ -99,7 +99,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:
@@ -136,7 +136,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:
@@ -169,7 +169,7 @@ jobs:
           verify
       - name: Upload test artifacts
         uses: ./.github/actions/collect-test-artifacts
-        if: always()
+        if: failure()
         with:
           name: "[Smoke] ${{ matrix.os }}"
   property-tests:
@@ -202,7 +202,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:
@@ -275,7 +275,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: