WIP CI

DataDog · Jan 30, 2025 · b9db45a · b9db45a
1 parent f091ea7
commit b9db45a
Showing 1 changed file with 60 additions and 32 deletions.
diff --git a/.github/workflows/test_workflow.yml b/.github/workflows/test_workflow.yml
@@ -61,19 +61,33 @@ jobs:
           ls -la ${GRAALVM_JDK_DIR}
       - name: Test
         run: |
+          set +e
           export TEST_COMMIT=${{ github.sha }}
           export TEST_CONFIGURATION=glibc/graalvm/${{ matrix.java_version }}-amd64
           export JAVA_TEST_HOME=$(pwd)/graalvm_${{ matrix.java_version }}_jdk
           export PATH=${JAVA_TEST_HOME}/bin:$PATH
-          ./gradlew clean build --no-daemon --parallel --build-cache --no-watch-fs
-      - name: Upload Logs
-        uses: actions/upload-artifact@v4
-        if: always()
+          ./gradlew -PkeepJFRs=true clean build --no-daemon --parallel --build-cache --no-watch-fs
+          EXIT_CODE=$?
+          rm -rf $JAVA_TEST_HOME
+          if [ $EXIT_CODE -ne 0 ]; then
+            echo "graalvm-${{ matrix.java_version }}-${{ matrix.config }}-amd64" >> failures_graalvm-${{ matrix.java_version }}-${{ matrix.config }}-amd64.txt
+            exit 1
+          fi
+      - uses: actions/upload-artifact@v4
+        if: success()
         with:
-          name: graalvm-${{ matrix.java_version }}-logs-amd64
-          path: |
-            build/reports/tests/
-            build/tmp/
+          name: graalvm-${{ matrix.java_version }}-${{ matrix.config }}-amd64
+          path: build/
+      - uses: actions/upload-artifact@v4
+        if: failure()
+        with:
+          name: failures-graalvm-${{ matrix.java_version }}-${{ matrix.config }}-amd64
+          path: failures_graalvm-${{ matrix.java_version }}-${{ matrix.config }}-amd64.txt
+      - uses: actions/upload-artifact@v4
+        if: failure()
+        with:
+          name: recordings-graalvm-${{ matrix.java_version }}-${{ matrix.config }}-amd64
+          path: /tmp/recordings
 
   test-linux-glibc-graalvm-aarch64:
     strategy:
@@ -128,19 +142,33 @@ jobs:
           ls -la ${GRAALVM_JDK_DIR}
       - name: Test
         run: |
+          set +e
           export TEST_COMMIT=${{ github.sha }}
           export TEST_CONFIGURATION=glibc/graalvm/${{ matrix.java_version }}-arm64
           export JAVA_TEST_HOME=$(pwd)/graalvm_${{ matrix.java_version }}_jdk
           export PATH=${JAVA_TEST_HOME}/bin:$PATH
-          ./gradlew clean build --no-daemon --parallel --build-cache --no-watch-fs
-      - name: Upload Logs
-        uses: actions/upload-artifact@v4
-        if: always()
+          ./gradlew -PkeepJFRs=true clean build --no-daemon --parallel --build-cache --no-watch-fs
+          EXIT_CODE=$?
+          rm -rf $JAVA_TEST_HOME
+          if [ $EXIT_CODE -ne 0 ]; then
+            echo "graalvm-${{ matrix.java_version }}-${{ matrix.config }}-aarch64" >> failures_graalvm-${{ matrix.java_version }}-${{ matrix.config }}-aarch64.txt
+            exit 1
+          fi
+      - uses: actions/upload-artifact@v4
+        if: success()
         with:
-          name: graalvm-${{ matrix.java_version }}-logs-aarch64
-          path: |
-            build/reports/tests/
-            build/tmp/
+          name: graalvm-${{ matrix.java_version }}-${{ matrix.config }}-aarch64
+          path: build/
+      - uses: actions/upload-artifact@v4
+        if: failure()
+        with:
+          name: failures-graalvm-${{ matrix.java_version }}-${{ matrix.config }}-aarch64
+          path: failures_graalvm-${{ matrix.java_version }}-${{ matrix.config }}-aarch64.txt
+      - uses: actions/upload-artifact@v4
+        if: failure()
+        with:
+          name: recordings-graalvm-${{ matrix.java_version }}-${{ matrix.config }}-aarch64
+          path: /tmp/recordings
 
   test-linux-glibc-amd64:
     strategy:
@@ -240,12 +268,12 @@ jobs:
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: failures
+          name: failures-${{ matrix.java_version }}-${{ matrix.config }}-amd64
           path: failures_glibc-${{ matrix.java_version }}-${{ matrix.config }}-amd64.txt
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: recordings
+          name: recordings-${{ matrix.java_version }}-${{ matrix.config }}-amd64
           path: /tmp/recordings
 
   test-linux-glibc-aarch64:
@@ -343,12 +371,12 @@ jobs:
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: failures
+          name: failures-${{ matrix.java_version }}-${{ matrix.config }}-aarch64
           path: failures_glibc-${{ matrix.java_version }}-${{ matrix.config }}-aarch64.txt
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: recordings
+          name: recordings-${{ matrix.java_version }}-${{ matrix.config }}-aarch64
           path: /tmp/recordings
 
   test-linux-glibc-j9-amd64:
@@ -437,12 +465,12 @@ jobs:
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: failures
+          name: failures-j9-${{ matrix.java_version }}-${{ matrix.config }}-amd64
           path: failures_glibc-j9-${{ matrix.java_version }}-${{ matrix.config }}-amd64.txt
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: recordings
+          name: recordings-j9-${{ matrix.java_version }}-${{ matrix.config }}-amd64
           path: /tmp/recordings
 
   test-linux-glibc-j9-aarch64:
@@ -534,12 +562,12 @@ jobs:
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: failures
+          name: failures-j9-${{ matrix.java_version }}-${{ matrix.config }}-aarch64
           path: failures_glibc-j9-${{ matrix.java_version }}-${{ matrix.config }}-aarch64.txt
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: recordings
+          name: recordings-j9-${{ matrix.java_version }}-${{ matrix.config }}-aarch64
           path: /tmp/recordings
 
   test-linux-glibc-oracle8:
@@ -639,12 +667,12 @@ jobs:
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: failures
+          name: failures-oracle-jdk8-${{ matrix.config }}
           path: failures_glibc-oracle8-${{ matrix.config }}.txt
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: recordings
+          name: recordings-oracle-jdk8-${{ matrix.config }}
           path: /tmp/recordings
 
   test-linux-musl-amd64:
@@ -760,12 +788,12 @@ jobs:
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: failures
+          name: failures-musl-${{ matrix.java_version }}-${{ matrix.config }}-amd64
           path: failures_musl-${{ matrix.java_version }}-${{ matrix.config }}-amd64.txt
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: recordings
+          name: recordings-musl-${{ matrix.java_version }}-${{ matrix.config }}-amd64
           path: /tmp/recordings
 
   test-linux-glibc-zing-amd64:
@@ -888,12 +916,12 @@ jobs:
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: failures
+          name: failures-zing-${{ matrix.java_version }}-${{ matrix.config }}-amd64
           path: failures_zing-${{ matrix.java_version }}-${{ matrix.config }}-amd64.txt
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: recordings
+          name: recordings-zing-${{ matrix.java_version }}-${{ matrix.config }}-amd64
           path: /tmp/recordings
 
   test-linux-glibc-zing-aarch64:
@@ -1018,10 +1046,10 @@ jobs:
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: failures
+          name: failures-zing-${{ matrix.java_version }}-${{ matrix.config }}-aarch64
           path: failures_zing-${{ matrix.java_version }}-${{ matrix.config }}-aarch64.txt
       - uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: recordings
+          name: recordings-zing-${{ matrix.java_version }}-${{ matrix.config }}-aarch64
           path: /tmp/recordings