From: Jérémie Galarneau Date: Thu, 22 Apr 2021 16:32:26 +0000 (-0400) Subject: Fix: tests: health thread stall: only stop consumerd when required X-Git-Tag: v2.13.0-rc1~23 X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=commitdiff_plain;h=5ff449ff970b67e444b4b402c3901270cf6424e1;ds=sidebyside Fix: tests: health thread stall: only stop consumerd when required Since a0f8e3109, stop_lttng_consumerd will report a failure when there is no consumer daemon to kill. This fix ensures it is only invoked for tests that launch a consumer daemon. Signed-off-by: Jérémie Galarneau Change-Id: I6831fbe7563d2e1804f10588494d126fbb4202ff --- diff --git a/tests/regression/tools/health/test_health.sh b/tests/regression/tools/health/test_health.sh index 4a99c2ea4..7417ba7a4 100644 --- a/tests/regression/tools/health/test_health.sh +++ b/tests/regression/tools/health/test_health.sh @@ -8,7 +8,7 @@ UST_EVENT_NAME="tp:tptest" KERNEL_EVENT_NAME="sched_switch" CHANNEL_NAME="testchan" HEALTH_CHECK_BIN="health_check" -NUM_TESTS=106 +NUM_TESTS=99 SLEEP_TIME=30 source $TESTDIR/utils/utils.sh @@ -126,7 +126,10 @@ function test_health # it may have been killed volountarily by testpoint. stop_lttng_relayd_notap $KILL_SIGNAL fi - stop_lttng_consumerd $KILL_SIGNAL + + if [ ${test_consumerd} -eq 1 ]; then + stop_lttng_consumerd $KILL_SIGNAL + fi stop_lttng_sessiond $KILL_SIGNAL unset LTTNG_TESTPOINT_ENABLE