(polaris) branch main updated: CI: simplify upload-artifacts call + only on failure (#3627)

[snazy]

This is an automated email from the ASF dual-hosted git repository.

snazy pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/main by this push:
     new 16a0e314c CI: simplify upload-artifacts call + only on failure (#3627)
16a0e314c is described below

commit 16a0e314cfd2f103be87e21881e32e065ca1c474
Author: Robert Stupp <sn...@snazy.de>
AuthorDate: Tue Feb 3 11:02:35 2026 +0100

    CI: simplify upload-artifacts call + only on failure (#3627)
---
 .github/workflows/gradle.yml | 28 ++++++++++------------------
 1 file changed, 10 insertions(+), 18 deletions(-)

diff --git a/.github/workflows/gradle.yml b/.github/workflows/gradle.yml
index f2a31ddc3..69e25911e 100644
--- a/.github/workflows/gradle.yml
+++ b/.github/workflows/gradle.yml
@@ -71,10 +71,10 @@ jobs:
         uses: ./.github/actions/ci-incr-build-cache-save
       - name: Archive test results
         uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f 
# v6
-        if: always()
+        if: failure()
         with:
           name: upload-unit-test-artifacts
-          path: |
+          path: &test-archive-path |
             **/build/test-results/**
             **/build/reports/tests/**
 
@@ -105,12 +105,10 @@ jobs:
         uses: ./.github/actions/ci-incr-build-cache-save
       - name: Archive test results
         uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f 
# v6
-        if: always()
+        if: failure()
         with:
           name: upload-quarkus-runtime-test-artifacts
-          path: |
-            **/build/test-results/**
-            **/build/reports/tests/**
+          path: *test-archive-path
 
   # Durations (approximates from GH CI):
   # - :polaris-runtime-service:intTest 15m
@@ -137,12 +135,10 @@ jobs:
         uses: ./.github/actions/ci-incr-build-cache-save
       - name: Archive test results
         uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f 
# v6
-        if: always()
+        if: failure()
         with:
           name: upload-quarkus-runtime-inttest-artifacts
-          path: |
-            **/build/test-results/**
-            **/build/reports/tests/**
+          path: *test-archive-path
 
   # Durations (approximates from GH CI):
   # - :polaris-admin:test 6m
@@ -169,12 +165,10 @@ jobs:
         uses: ./.github/actions/ci-incr-build-cache-save
       - name: Archive test results
         uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f 
# v6
-        if: always()
+        if: failure()
         with:
           name: upload-quarkus-admin-test-artifacts
-          path: |
-            **/build/test-results/**
-            **/build/reports/tests/**
+          path: *test-archive-path
 
   # Durations (approximates from GH CI):
   # - :polaris-spark-integration-*:intTest 6m (2x)
@@ -208,12 +202,10 @@ jobs:
         uses: ./.github/actions/ci-incr-build-cache-save
       - name: Archive test results
         uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f 
# v6
-        if: always()
+        if: failure()
         with:
           name: upload-integration-test-artifacts
-          path: |
-            **/build/test-results/**
-            **/build/reports/tests/**
+          path: *test-archive-path
 
   store-gradle-cache:
     name: Store Gradle Cache