Don't upload surefire reports and container logs to artifacts unless …

…jobs fail
apache · Mar 30, 2022 · 6a8656d · 6a8656d
1 parent 9951dbd
commit 6a8656d
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/.github/workflows/pulsar-ci.yaml b/.github/workflows/pulsar-ci.yaml
@@ -236,7 +236,7 @@ jobs:
 
       - name: Upload Surefire reports
         uses: actions/upload-artifact@v2
-        if: ${{ always() && needs.changed_files_job.outputs.docs_only != 'true' }}
+        if: ${{ !success() && needs.changed_files_job.outputs.docs_only != 'true' }}
         with:
           name: Unit-${{ matrix.group }}-surefire-reports
           path: surefire-reports
@@ -434,14 +434,14 @@ jobs:
 
       - name: Upload Surefire reports
         uses: actions/upload-artifact@v2
-        if: ${{ always() && needs.changed_files_job.outputs.docs_only != 'true' }}
+        if: ${{ !success() && needs.changed_files_job.outputs.docs_only != 'true' }}
         with:
           name: Integration-${{ matrix.group }}-surefire-reports
           path: surefire-reports
 
       - name: Upload container logs
         uses: actions/upload-artifact@v2
-        if: ${{ always() && needs.changed_files_job.outputs.docs_only != 'true' }}
+        if: ${{ !success() && needs.changed_files_job.outputs.docs_only != 'true' }}
         continue-on-error: true
         with:
           name: Integration-${{ matrix.group }}-container-logs
@@ -667,15 +667,15 @@ jobs:
 
       - name: Upload container logs
         uses: actions/upload-artifact@v2
-        if: ${{ always() && needs.changed_files_job.outputs.docs_only != 'true' }}
+        if: ${{ !success() && needs.changed_files_job.outputs.docs_only != 'true' }}
         continue-on-error: true
         with:
           name: System-${{ matrix.group }}-container-logs
           path: tests/integration/target/container-logs
 
       - name: Upload Surefire reports
         uses: actions/upload-artifact@v2
-        if: ${{ always() && needs.changed_files_job.outputs.docs_only != 'true' }}
+        if: ${{ !success() && needs.changed_files_job.outputs.docs_only != 'true' }}
         with:
           name: System-${{ matrix.name }}-surefire-reports
           path: surefire-reports