X-Git-Url: https://git.lttng.org/?a=blobdiff_plain;f=tests%2Fstress%2Ftest_multi_sessions_per_uid_5app_streaming;h=8800f6f233679e44d7ffb2d5d713cc078f901ca9;hb=c1896e2d809e0d32d2d9b8a840755de6d5d70eaf;hp=d1d192c029bcaf13a91513df5f0e80a2fc28cb1d;hpb=f7169e41979bb8a57ecf2ff3683fefa74e05179b;p=lttng-tools.git

diff --git a/tests/stress/test_multi_sessions_per_uid_5app_streaming b/tests/stress/test_multi_sessions_per_uid_5app_streaming
index d1d192c02..8800f6f23 100755
--- a/tests/stress/test_multi_sessions_per_uid_5app_streaming
+++ b/tests/stress/test_multi_sessions_per_uid_5app_streaming
@@ -5,8 +5,8 @@
 # SPDX-License-Identifier: LGPL-2.1-only
 #
 
-CURDIR=$(dirname $0)/
-TESTDIR=$CURDIR/..
+CURDIR=$(dirname "$0")/
+TESTDIR="$CURDIR/.."
 LAUNCH_APP="launch_ust_app"
 SESSION_NAME="stress"
 EVENT_NAME="tp:tptest"
@@ -18,32 +18,34 @@ NR_SESSION=5
 NR_LOOP=1000
 COREDUMP_FILE=$(cat /proc/sys/kernel/core_pattern)
 NUM_TESTS=16
-APPS_PID=
+APP_PIDS=()
 
 TEST_DESC="Stress test - $NR_SESSION sessions per UID streaming with $NR_APP apps"
 
-source $TESTDIR/utils/utils.sh
+# shellcheck source-path=SCRIPTDIR/../
+source "$TESTDIR/utils/utils.sh"
 
 # MUST set TESTDIR before calling those functions
 
 function enable_channel_per_uid()
 {
-    local sess_name=$1
-    local channel_name=$2
+	local sess_name=$1
+	local channel_name=$2
 
-    $TESTDIR/../src/bin/lttng/$LTTNG_BIN enable-channel --buffers-uid -u $channel_name -s $sess_name >/dev/null 2>&1
-    ok $? "Enable channel $channel_name per UID for session $sess_name"
+	"$TESTDIR/../src/bin/lttng/$LTTNG_BIN" enable-channel --buffers-uid -u "$channel_name" -s "$sess_name" >/dev/null 2>&1
+	ok $? "Enable channel $channel_name per UID for session $sess_name"
 }
 
 function check_sessiond()
 {
-	if [ -z "$(lttng_pgrep lt-lttng-sessiond)" ]; then
-		local str_date=$(date +%H%M%S-%d%m%Y)
+	local str_date
+	if [ -z "$(lttng_pgrep lttng-sessiond)" ]; then
+		str_date=$(date +%H%M%S-%d%m%Y)
 
 		diag "!!!The session daemon died unexpectedly!!!"
-		mv $LOG_FILE_SESSIOND $LOG_FILE_SESSIOND-$str_date
-		if [ -e $COREDUMP_FILE ]; then
-			mv $COREDUMP_FILE $COREDUMP_FILE-$str_date
+		mv $LOG_FILE_SESSIOND "$LOG_FILE_SESSIOND-$str_date"
+		if [ -e "$COREDUMP_FILE" ]; then
+			mv "$COREDUMP_FILE" "$COREDUMP_FILE-$str_date"
 		fi
 		exit 1
 	fi
@@ -51,13 +53,14 @@ function check_sessiond()
 
 function check_relayd()
 {
-	if [ -z "$(lttng_pgrep lt-lttng-relayd)" ]; then
-		local str_date=$(date +%H%M%S-%d%m%Y)
+	local str_date
+	if [ -z "$(lttng_pgrep lttng-relayd)" ]; then
+		str_date=$(date +%H%M%S-%d%m%Y)
 
 		diag "!!!The relay daemon died unexpectedly!!!"
-		mv $LOG_FILE_RELAYD $LOG_FILE_RELAYD-$str_date
-		if [ -e $COREDUMP_FILE ]; then
-			mv $COREDUMP_FILE $COREDUMP_FILE-$str_date
+		mv $LOG_FILE_RELAYD "$LOG_FILE_RELAYD-$str_date"
+		if [ -e "$COREDUMP_FILE" ]; then
+			mv "$COREDUMP_FILE" "$COREDUMP_FILE-$str_date"
 		fi
 		exit 1
 	fi
@@ -65,16 +68,15 @@ function check_relayd()
 
 function start_sessiond()
 {
-	validate_kernel_version
-	if [ $? -ne 0 ]; then
+	if ! validate_kernel_version ; then
 		fail "Start session daemon"
 		BAIL_OUT "*** Kernel too old for session daemon tests ***"
 	fi
 
-	if [ -z $(lttng_pgrep lt-$SESSIOND_BIN) ]; then
+	if [ -z "$(lttng_pgrep lt-$SESSIOND_BIN)" ]; then
 		# We have to start it like this so the ulimit -c is used by this
 		# process. Also, we collect any error message printed out.
-		$TESTDIR/../src/bin/lttng-sessiond/$SESSIOND_BIN --quiet --background --consumerd32-path="$TESTDIR/../src/bin/lttng-consumerd/lttng-consumerd" --consumerd64-path="$TESTDIR/../src/bin/lttng-consumerd/lttng-consumerd" >$LOG_FILE_SESSIOND 2>&1
+		"$TESTDIR/../src/bin/lttng-sessiond/$SESSIOND_BIN" --quiet --background --consumerd32-path="$TESTDIR/../src/bin/lttng-consumerd/lttng-consumerd" --consumerd64-path="$TESTDIR/../src/bin/lttng-consumerd/lttng-consumerd" >$LOG_FILE_SESSIOND 2>&1
 		status=$?
 		ok $status "Start session daemon"
 	fi
@@ -84,35 +86,36 @@ function start_relayd
 {
 	local opt=$1
 
-	if [ -z $(lttng_pgrep lt-$RELAYD_BIN) ]; then
-		$TESTDIR/../src/bin/lttng-relayd/$RELAYD_BIN $opt >$LOG_FILE_RELAYD 2>&1 &
+	if [ -z "$(lttng_pgrep lt-$RELAYD_BIN)" ]; then
+		"$TESTDIR/../src/bin/lttng-relayd/$RELAYD_BIN" "$opt" >"$LOG_FILE_RELAYD" 2>&1 &
 		ok $? "Start lttng-relayd (opt: \"$opt\")"
 	fi
 }
 
 test_stress()
 {
+	# shellcheck disable=SC2034
 	for b in $(seq 1 $NR_LOOP); do
 		for a in $(seq 1 $NR_SESSION); do
-			create_lttng_session_uri $SESSION_NAME-$a net://localhost
+			create_lttng_session_uri $SESSION_NAME-"$a" net://localhost
 			check_sessiond
 			check_relayd
-			enable_channel_per_uid $SESSION_NAME-$a $CHANNEL_NAME
+			enable_channel_per_uid $SESSION_NAME-"$a" $CHANNEL_NAME
 			check_sessiond
 			check_relayd
-			enable_ust_lttng_event_ok $SESSION_NAME-$a $EVENT_NAME
+			enable_ust_lttng_event_ok $SESSION_NAME-"$a" $EVENT_NAME
 			check_sessiond
 			check_relayd
-			start_lttng_tracing_ok $SESSION_NAME-$a
+			start_lttng_tracing_ok $SESSION_NAME-"$a"
 			check_sessiond
 			check_relayd
 		done
 
 		for a in $(seq 1 $NR_SESSION); do
-			stop_lttng_tracing_ok $SESSION_NAME-$a
+			stop_lttng_tracing_ok $SESSION_NAME-"$a"
 			check_sessiond
 			check_relayd
-			destroy_lttng_session_ok $SESSION_NAME-$a
+			destroy_lttng_session_ok $SESSION_NAME-"$a"
 			check_sessiond
 			check_relayd
 		done
@@ -124,12 +127,12 @@ test_stress()
 function cleanup()
 {
 	diag "Cleaning up!"
-	for p in ${APPS_PID}; do
-		kill -s SIGKILL ${p}
-		wait ${p} 2>/dev/null
-	done
-	APPS_PID=
+	kill -s SIGKILL "${APP_PIDS[@]}"
+	wait "${APP_PIDS[@]}" 2>/dev/null
+	APP_PIDS=()
+	# shellcheck disable=SC2119
 	stop_lttng_sessiond
+	# shellcheck disable=SC2119
 	stop_lttng_relayd
 }
 
@@ -158,8 +161,8 @@ start_sessiond
 diag "Starting applications launcher"
 
 # Start NR_APP applications script that will spawn apps non stop.
-./$TESTDIR/stress/$LAUNCH_APP $NR_APP &
-APPS_PID="${APPS_PID} ${!}"
+"./$TESTDIR/stress/$LAUNCH_APP" $NR_APP &
+APP_PIDS+=(${!})
 
 test_stress
 out=$?
@@ -169,6 +172,6 @@ if [ $out -ne 0 ]; then
 fi
 
 cleanup
-rm -rf $TRACE_PATH
+rm -rf "${TRACE_PATH:?}/"
 rm $LOG_FILE_SESSIOND $LOG_FILE_RELAYD
 exit 0