diff options
| -rw-r--r-- | test/Makefile | 10 | ||||
| -rwxr-xr-x | test/scripts/run_in_venv_with_cleanup.sh (renamed from test/scripts/run_with_cleanup.sh) | 17 | ||||
| -rwxr-xr-x | test/scripts/setsid_wrapper.sh | 12 |
3 files changed, 36 insertions, 3 deletions
diff --git a/test/Makefile b/test/Makefile index d48a6b5abd8..33779dcebd9 100644 --- a/test/Makefile +++ b/test/Makefile @@ -13,6 +13,14 @@ else VPP_PIDS=$(shell pgrep -d, -x vpp_main) endif +ifeq ($(DEBUG),gdb) +FORCE_FOREGROUND=1 +else ifeq ($(DEBUG),gdbserver) +FORCE_FOREGROUND=1 +else +FORCE_FOREGROUND=0 +endif + verify-no-running-vpp: @if [ "$(VPP_PIDS)" != "" ]; then \ echo; \ @@ -76,7 +84,7 @@ $(PAPI_INSTALL_DONE): $(PIP_PATCH_DONE) @touch $@ define retest-func - @bash -c "source $(PYTHON_VENV_PATH)/bin/activate && setsid scripts/run_with_cleanup.sh python run_tests.py -d $(TEST_DIR) $(UNITTEST_EXTRA_OPTS)" + @scripts/setsid_wrapper.sh $(FORCE_FOREGROUND) $(PYTHON_VENV_PATH)/bin/activate python run_tests.py -d $(TEST_DIR) $(UNITTEST_EXTRA_OPTS) endef .PHONY: sanity diff --git a/test/scripts/run_with_cleanup.sh b/test/scripts/run_in_venv_with_cleanup.sh index dcaa58b4498..541f584ac45 100755 --- a/test/scripts/run_with_cleanup.sh +++ b/test/scripts/run_in_venv_with_cleanup.sh @@ -17,9 +17,22 @@ atexit() { exit $rv } -trap "atexit" SIGINT SIGTERM +trap "atexit;" SIGINT SIGTERM + +FORCE_FOREGROUND=$1 +shift + +source $1 +shift + +if [[ "${FORCE_FOREGROUND}" == "1" ]] +then + $* +else + $* & + wait +fi -$* rv=$? atexit exit $rv diff --git a/test/scripts/setsid_wrapper.sh b/test/scripts/setsid_wrapper.sh new file mode 100755 index 00000000000..e18b6ad5947 --- /dev/null +++ b/test/scripts/setsid_wrapper.sh @@ -0,0 +1,12 @@ +#!/bin/bash + +if [[ "$1" == "1" ]] +then + setsid scripts/run_in_venv_with_cleanup.sh $* + pid=$! +else + setsid scripts/run_in_venv_with_cleanup.sh $* & + pid=$! + trap "echo setsid_wrapper.sh: got signal, killing child pid ${pid}; kill ${pid}; sleep .1;" SIGINT SIGTERM + wait +fi |