cleaning up the test workflow

nasa · Sep 23, 2024 · 6a869a7 · 6a869a7
1 parent 7e3547c
commit 6a869a7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -21,9 +21,6 @@ jobs:
 
     - name: Install MPI
       uses: mpi4py/setup-mpi@v1
-    #   run: |
-    #     sudo apt-get update
-    #     sudo apt-get install -y mpich libmpich-dev
 
     - name: Install dependencies
       run: |
@@ -36,9 +33,11 @@ jobs:
     - name: Set pythonpath
       run: |
         echo "PYTHONPATH=`pwd`" >> $GITHUB_ENV
-        echo "OMP_NUM_THREADS=1" >> $GITHUB_ENV
         echo "RDMAV_FORK_SAFE=1" >> $GITHUB_ENV
 
+    - name: Set safe forking var
+      run: |
+        echo "RDMAV_FORK_SAFE=1" >> $GITHUB_ENV
 
     - name: Build bingocpp
       run: |

diff --git a/tests/.run_tests.sh b/tests/.run_tests.sh
@@ -10,7 +10,7 @@ then
 fi
 echo "found $MPI_EXEC"
 
-RUN_MODE=${1-"normal"}
+RUN_MODE=${1-"coverage"}
 echo "Running tests in $RUN_MODE mode"
 
 # run mpi tests
@@ -21,10 +21,10 @@ do
     echo "Running mpitest: $i in $RUN_MODE mode"
     if [ $RUN_MODE == "coverage" ]
     then
-      $MPI_EXEC -n 2 coverage run --parallel-mode --source=bingo $i
+      $MPI_EXEC -n 3 coverage run --parallel-mode --source=bingo $i
     elif [ $RUN_MODE == "normal" ]
     then
-      $MPI_EXEC -n 2 --bind-to none python $i
+      $MPI_EXEC -n 3 python $i
     fi
   fi
 done