Post test CI improvements (#2931)

zeroc-ice · Oct 18, 2024 · a024737 · a024737
1 parent ffa3363
commit a024737
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 15 deletions.
diff --git a/.github/actions/test/action.yml b/.github/actions/test/action.yml
@@ -24,17 +24,3 @@ runs:
       run: python allTests.py --debug --all --continue --workers=4 --export-xml=test-report.xml ${{ inputs.flags }}
       shell: powershell
       if: runner.os == 'Windows'
-
-    - name: Upload test logs
-      uses: actions/upload-artifact@v4
-      with:
-        name: test-logs-${{ matrix.config }}-${{ matrix.os }}
-        path: ${{ inputs.working_directory }}/**/*.log
-        if-no-files-found: ignore
-      if: always()
-
-    - name: Test Summary
-      uses: test-summary/action@v2
-      with:
-        paths: "${{ inputs.working_directory }}/test-report.xml"
-      if: always()
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -101,11 +101,25 @@ jobs:
 
       - name: Test ${{ matrix.config }} on ${{ matrix.os }}
         uses: ./.github/actions/test
-        timeout-minutes: 90
+        timeout-minutes: 30
         with:
           working_directory: ${{ matrix.working_directory || '.' }}
           # See:
           # - https://github.com/zeroc-ice/ice/issues/1653 IceGrid/replication
           flags: "--rfilter=IceGrid/replication ${{ matrix.test_flags }}"
         # Don't test matlab on Windows (see setup-dependencies/action.yml)
         if: matrix.config != 'matlab' || runner.os != 'Windows'
+
+      - name: Upload test logs
+        uses: actions/upload-artifact@v4
+        with:
+          name: test-logs-${{ matrix.config }}-${{ matrix.os }}
+          path: ${{ matrix.working_directory || '.' }}/**/*.log
+          if-no-files-found: ignore
+        if: always()
+
+      # - name: Test Summary
+      #   uses: test-summary/action@v2
+      #   with:
+      #     paths: "${{ matrix.working_directory || '.' }}/test-report.xml"
+      #   if: always()