tests: upload ci logs (#741)

Signed-off-by: dongmen <[email protected]>
pingcap · Dec 30, 2024 · 882481a · 882481a
1 parent e78f686
commit 882481a
Showing 1 changed file with 18 additions and 20 deletions.
diff --git a/.github/workflows/integration_test_mysql.yaml b/.github/workflows/integration_test_mysql.yaml
@@ -71,27 +71,25 @@ jobs:
         run: |
           export TICDC_NEWARCH=true && make integration_test CASE=changefeed_reconstruct
 
-      # only upload logs of the last case
-      # - name: Copy logs to hack permission
-      #   if: ${{ always() }}
-      #   run: |
-      #     DIR=$(sudo find /tmp/tidb_cdc_test/ -type d -name 'sync_diff' -exec dirname {} \;)
-      #     CASE=$(basename $DIR)
-      #     mkdir -p ./logs/$CASE
-      #     cat $DIR/stdout.log
-      #     tail -n 10 $DIR/cdc.log
-      #     sudo cp -r -L $DIR/{cdc.log,stdout.log,sync_diff} ./logs/$CASE/
-      #     sudo chown -R runner ./logs
-      #     sudo tar -czvf ./logs.tar.gz ./logs
-
+      - name: Copy logs to hack permission
+        if: ${{ always() }}
+        run: |
+          DIR=$(sudo find /tmp/tidb_cdc_test/ -type d -name 'cdc_data' -exec dirname {} \;)
+          CASE=$(basename $DIR)
+          mkdir -p ./logs/$CASE
+          cat $DIR/stdout.log
+          tail -n 10 $DIR/cdc.log
+          sudo cp -r -L $DIR/{*.log,sync_diff} ./logs/$CASE/
+          sudo chown -R runner ./logs
+          sudo tar -czvf ./logs.tar.gz ./logs
       # Update logs as artifact seems not stable, so we set `continue-on-error: true` here.
-      # - name: Upload logs
-      #   uses: actions/upload-artifact@v4
-      #   if: ${{ always() }}
-      #   with:
-      #     name: upstream-switch-logs
-      #     path: |
-      #       ./logs.tar.gz
+      - name: Upload logs
+        uses: actions/upload-artifact@v4
+        if: ${{ always() }}
+        with:
+          name: upstream-switch-logs
+          path: |
+            ./logs.tar.gz
 
   failover_e2e_test:
     runs-on: ubuntu-latest