More log and timeout in the CI (#249)

* more log? * timeout * typo * fu
argonne-lcf · Jul 2, 2024 · 9eb97fa · 9eb97fa
1 parent e83572b
commit 9eb97fa
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/.github/workflows/presubmit.yml b/.github/workflows/presubmit.yml
@@ -165,7 +165,7 @@ jobs:
           name: thapi-bin
       - name: Untar THAPI
         run: tar -xvf thapi.tar
-      - run: sudo apt update; sudo apt install -y lttng-tools liblttng-ust-dev ruby ruby-dev libprotobuf-dev libpocl2 clinfo bats
+      - run: sudo apt update; sudo apt install -y lttng-tools liblttng-ust-dev ruby ruby-dev libprotobuf-dev libpocl2 clinfo bats coreutils
       - run: sudo gem install babeltrace2 opencl_ruby_ffi
       - name: Load Babeltrace2
         run: |

diff --git a/integration_tests/mpi_sync_daemon.bats b/integration_tests/mpi_sync_daemon.bats
@@ -8,5 +8,6 @@ setup_file() {
 }
 
 @test "sync_daemon_${THAPI_SYNC_DAEMON}" {
-   THAPI_JOBID=0 $MPIRUN -n 2 ./integration_tests/sync_daemon_test/test.sh
+   export THAPI_JOBID=0
+   timeout 20s $MPIRUN -n 2 ./integration_tests/sync_daemon_test/test.sh
 }
diff --git a/integration_tests/sync_daemon_test/test.sh b/integration_tests/sync_daemon_test/test.sh
@@ -1,4 +1,5 @@
 #!/bin/bash
+set -euo pipefail
 # Usage: IPROF_BIN_DIR=/path/to/iprof/bin THAPI_SYNC_DAEMON=mpi|fs TEST_EXE=clinfo test.sh
 
 # Get base real-time signal number