[CELEBORN-1569] Upload unit-tests.log and test-reports when CI fails

### What changes were proposed in this pull request? ### Why are the changes needed? Missing detailed logs when SBT CI fails. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? GA Closes #2693 from cxzl25/CELEBORN-1569. Authored-by: sychen <sychen@ctrip.com> Signed-off-by: mingji <fengmingxiao.fmx@alibaba-inc.com>
apache · Aug 19, 2024 · 4a8723f · 4a8723f
1 parent b330b55
commit 4a8723f
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/.github/workflows/maven.yml b/.github/workflows/maven.yml
@@ -40,7 +40,9 @@ jobs:
       with:
         name: service-unit-test-log
         path: |
+          **/target/test-reports/**
           **/target/unit-tests.log
+
     - name: Upload coverage to Codecov
       if: matrix.java == 8
       uses: codecov/codecov-action@v3
@@ -77,6 +79,7 @@ jobs:
         with:
           name: spark-${{ matrix.spark }}-unit-test-log
           path: |
+            **/target/test-reports/**
             **/target/unit-tests.log
 
   spark3:
@@ -140,6 +143,7 @@ jobs:
       with:
         name: spark-${{ matrix.spark }}-unit-test-log
         path: |
+          **/target/test-reports/**
           **/target/unit-tests.log
 
   flink:
@@ -179,6 +183,7 @@ jobs:
         with:
           name: flink-${{ matrix.flink }}-unit-test-log
           path: |
+            **/target/test-reports/**
             **/target/unit-tests.log
 
   mr:
@@ -211,3 +216,5 @@ jobs:
           name: mr-unit-test-log
           path: |
             **/target/test/
+            **/target/test-reports/**
+            **/target/unit-tests.log
diff --git a/.github/workflows/sbt.yml b/.github/workflows/sbt.yml
@@ -59,6 +59,7 @@ jobs:
           name: service-java-${{ matrix.java }}-scala-${{ matrix.scala }}-unit-test-log
           path: |
               **/target/test-reports/**
+              **/target/unit-tests.log
 
   spark2:
     runs-on: ubuntu-22.04
@@ -90,7 +91,7 @@ jobs:
             name: spark-${{ matrix.spark }}-unit-test-log
             path: |
                 **/target/test-reports/**
-
+                **/target/unit-tests.log
   spark3:
     runs-on: ubuntu-22.04
     strategy:
@@ -193,6 +194,7 @@ jobs:
           name: spark-${{ matrix.spark }}-scala-${{ matrix.scala }}-unit-test-log
           path: |
               **/target/test-reports/**
+              **/target/unit-tests.log
 
   flink:
     runs-on: ubuntu-22.04
@@ -228,6 +230,7 @@ jobs:
             name: flink-${{ matrix.flink }}-unit-test-log
             path: |
                 **/target/test-reports/**
+                **/target/unit-tests.log
 
   mr:
     runs-on: ubuntu-22.04
@@ -255,3 +258,5 @@ jobs:
           name: mr-unit-test-log
           path: |
             **/target/test/
+            **/target/test-reports/**
+            **/target/unit-tests.log