testing/test_runner: Only set errexit during setup

We rely on `set -o errexit` to abort tests if any of the setup commands fail. However, we don't want this turned on during test execution, since we rely on explicit checks using check_run() to check the output of certain commands in the tests. So move the errexit to cover only the setup function. Signed-off-by: Toke Høiland-Jørgensen <[email protected]>
xdp-project · Aug 5, 2024 · c580e7c · c580e7c
1 parent 5c7239b
commit c580e7c
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/lib/testing/test_runner.sh b/lib/testing/test_runner.sh
@@ -8,7 +8,6 @@
 # Date:     26 May 2020
 # Copyright (c) 2020 Red Hat
 
-set -o errexit
 set -o nounset
 umask 077
 
@@ -336,6 +335,8 @@ setup()
 {
     local nsname
 
+    set -o errexit
+
     check_prereq
 
     for i in $(seq $NUM_NS); do
@@ -344,6 +345,8 @@ setup()
         NS_NAMES+=($nsname)
     done
 
+    set +o errexit
+
     NS=$nsname
 }