projects
/
lttng-tools.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Tests: Avoid looping on app PIDs during cleanup
[lttng-tools.git]
/
tests
/
stress
/
test_multi_sessions_per_uid_10app
diff --git
a/tests/stress/test_multi_sessions_per_uid_10app
b/tests/stress/test_multi_sessions_per_uid_10app
index 68cfc457242d9c5c9538032f74eae301a340239e..5ddc6be64eb5675ceaa88bc006c7a211d2a42ab6 100755
(executable)
--- a/
tests/stress/test_multi_sessions_per_uid_10app
+++ b/
tests/stress/test_multi_sessions_per_uid_10app
@@
-5,8
+5,8
@@
# SPDX-License-Identifier: LGPL-2.1-only
#
# 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"
LAUNCH_APP="launch_ust_app"
SESSION_NAME="stress"
EVENT_NAME="tp:tptest"
@@
-17,32
+17,34
@@
NR_APP=10
NR_SESSION=5
NR_LOOP=1000
COREDUMP_FILE=$(cat /proc/sys/kernel/core_pattern)
NR_SESSION=5
NR_LOOP=1000
COREDUMP_FILE=$(cat /proc/sys/kernel/core_pattern)
-APP
S_PID=
+APP
_PIDS=()
TEST_DESC="Stress test - $NR_SESSION sessions per UID with $NR_APP apps"
TEST_DESC="Stress test - $NR_SESSION sessions per UID 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()
{
# 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()
{
}
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!!!"
diag "!!!The session daemon died unexpectedly!!!"
- mv $LOG_FILE
$LOG_FILE-$str_date
- if [ -e
$COREDUMP_FILE
]; then
- mv
$COREDUMP_FILE $COREDUMP_FILE-$str_date
+ mv $LOG_FILE
"$LOG_FILE-$str_date"
+ if [ -e
"$COREDUMP_FILE"
]; then
+ mv
"$COREDUMP_FILE" "$COREDUMP_FILE-$str_date"
fi
exit 1
fi
fi
exit 1
fi
@@
-50,16
+52,15
@@
function check_sessiond()
function start_sessiond()
{
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
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 $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.
# 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 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 2>&1
status=$?
ok $status "Start session daemon"
fi
status=$?
ok $status "Start session daemon"
fi
@@
-67,22
+68,23
@@
function start_sessiond()
test_stress()
{
test_stress()
{
+ # shellcheck disable=SC2034
for b in $(seq 1 $NR_LOOP); do
for a in $(seq 1 $NR_SESSION); do
for b in $(seq 1 $NR_LOOP); do
for a in $(seq 1 $NR_SESSION); do
- create_lttng_session_ok $SESSION_NAME-
$a $TRACE_PATH
+ create_lttng_session_ok $SESSION_NAME-
"$a" "$TRACE_PATH"
check_sessiond
check_sessiond
- enable_channel_per_uid $SESSION_NAME-
$a
$CHANNEL_NAME
+ enable_channel_per_uid $SESSION_NAME-
"$a"
$CHANNEL_NAME
check_sessiond
check_sessiond
- enable_ust_lttng_event_ok $SESSION_NAME-
$a
$EVENT_NAME
+ enable_ust_lttng_event_ok $SESSION_NAME-
"$a"
$EVENT_NAME
check_sessiond
check_sessiond
- start_lttng_tracing_ok $SESSION_NAME-
$a
+ start_lttng_tracing_ok $SESSION_NAME-
"$a"
check_sessiond
done
for a in $(seq 1 $NR_SESSION); do
check_sessiond
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_sessiond
- destroy_lttng_session_ok $SESSION_NAME-
$a
+ destroy_lttng_session_ok $SESSION_NAME-
"$a"
check_sessiond
done
done
check_sessiond
done
done
@@
-93,18
+95,17
@@
test_stress()
function cleanup()
{
diag "Cleaning up!"
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
}
function sighandler()
{
cleanup
stop_lttng_sessiond
}
function sighandler()
{
cleanup
- rm
$LOG_FILE
+ rm
"$LOG_FILE"
full_cleanup
}
full_cleanup
}
@@
-123,8
+124,8
@@
start_sessiond
diag "Starting applications"
# Start NR_APP applications script that will spawn apps non stop.
diag "Starting applications"
# Start NR_APP applications script that will spawn apps non stop.
-
./$TESTDIR/stress/$LAUNCH_APP
$NR_APP &
-APP
S_PID="${APPS_PID} ${!}"
+
"./$TESTDIR/stress/$LAUNCH_APP"
$NR_APP &
+APP
_PIDS+=(${!})
TRACE_PATH=$(mktemp -d -t tmp.test_multi_sess_per_uid_10app.XXXXXX)
TRACE_PATH=$(mktemp -d -t tmp.test_multi_sess_per_uid_10app.XXXXXX)
@@
-136,6
+137,6
@@
if [ $out -ne 0 ]; then
fi
cleanup
fi
cleanup
-rm -rf
$TRACE_PATH
+rm -rf
"${TRACE_PATH:?}/"
rm $LOG_FILE
exit 0
rm $LOG_FILE
exit 0
This page took
0.026854 seconds
and
4
git commands to generate.