Perform cleanup when necessary. Use rm -f to prevent interactive mode when files or folders to be deleted have permission issues (e.g root owned but running test under user).
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-jul...@efficios.com> --- .../tools/notification/test_notification_multi_app | 111 +++++++++++---------- 1 file changed, 56 insertions(+), 55 deletions(-) diff --git a/tests/regression/tools/notification/test_notification_multi_app b/tests/regression/tools/notification/test_notification_multi_app index d7f6319f..53d51685 100755 --- a/tests/regression/tools/notification/test_notification_multi_app +++ b/tests/regression/tools/notification/test_notification_multi_app @@ -40,7 +40,7 @@ DIR=$(readlink -f $TESTDIR) PAGE_SIZE=$(getconf PAGE_SIZE) -NUM_TESTS=62 +NUM_TESTS=60 source $TESTDIR/utils/utils.sh @@ -73,10 +73,11 @@ function start_client { function wait_for_message () { - local file_pattern=$1 - local message=$2 + local tmpdir=$1 + local file_pattern=$2 + local message=$3 - for file in $CURDIR/${file_pattern}*; do + for file in ${tmpdir}/${file_pattern}*; do while(true); do # Check for "error" message grep -q "error:" ${file} @@ -103,9 +104,10 @@ function wait_for_message () function print_errors () { - local file_pattern=$1 + local tmpdir=$1 + local file_pattern=$2 - for file in $CURDIR/${file_pattern}*; do + for file in ${tmpdir}/${file_pattern}*; do # Check for "error" message error_message=$(grep "error:" ${file}) if [[ "${error_message}x" != "x" ]]; then @@ -143,15 +145,13 @@ function test_multi_app () local low_output_file_pattern="low_app_output_file_" local high_output_file_pattern="high_app_output_file_" - local testpoint_base_path=$(readlink -f "$CURDIR/lttng.t_p_n_multi_app") + local tmpdir=$(mktemp -d) + local testpoint_base_path=$(readlink -f "$tmpdir/lttng.t_p_n_multi_app") local testpoint_pipe_path=$(mktemp -u "${testpoint_base_path}.XXXXXX") local nr_notification_expected=5 local nr_client_app=50 - - # Cleanup - rm ${CURDIR}/${low_output_file_pattern}* 2> /dev/null - rm ${CURDIR}/${high_output_file_pattern}* 2> /dev/null + local consumerd_pipe=() # Setup LTTNG_SESSIOND_ENV_VARS="LTTNG_TESTPOINT_ENABLE=1 CONSUMER_PAUSE_PIPE_PATH=${testpoint_pipe_path} LD_PRELOAD=${TESTPOINT}" @@ -175,19 +175,19 @@ function test_multi_app () # Fetch consumerd testpoint pipe information # This is needed since the testpoint create a pipe with the consumer type suffixed - for f in "$testpoint_base_path"*; do + for f in "$testpoint_pipe_path"*; do consumerd_pipe+=("$f") done for (( i = 0; i < $nr_client_app; i++ )); do - low_app_output_file=$CURDIR/${low_output_file_pattern}${i} - high_app_output_file=$CURDIR/${high_output_file_pattern}${i} + low_app_output_file=${tmpdir}/${low_output_file_pattern}${i} + high_app_output_file=${tmpdir}/${high_output_file_pattern}${i} start_client $low_app_output_file $SESSION_NAME $UST_CHANNEL_NAME LTTNG_DOMAIN_UST LOW RATIO 0.0 $nr_notification_expected start_client $high_app_output_file $SESSION_NAME $UST_CHANNEL_NAME LTTNG_DOMAIN_UST HIGH RATIO 0.420 $nr_notification_expected done - wait_for_message "${low_output_file_pattern}" "sync: ready" - wait_for_message "${high_output_file_pattern}" "sync: ready" + wait_for_message "${tmpdir}" "${low_output_file_pattern}" "sync: ready" + wait_for_message "${tmpdir}" "${high_output_file_pattern}" "sync: ready" # Test notification reception for (( i = 0; i < $nr_notification_expected; i++ )); do @@ -198,7 +198,7 @@ function test_multi_app () stop_consumerd "${pipe}" done - wait_for_message "${high_output_file_pattern}" "notification: high $i" + wait_for_message "${tmpdir}" "${high_output_file_pattern}" "notification: high $i" # Resume consumerd for pipe in "${consumerd_pipe[@]}"; do @@ -208,7 +208,7 @@ function test_multi_app () stop_lttng_tracing $SESSION_NAME # Check for notifications reception - wait_for_message "${low_output_file_pattern}" "notification: low $i" + wait_for_message "${tmpdir}" "${low_output_file_pattern}" "notification: low $i" ret=$? ok $ret "Notifications $i received" if [[ $ret -ne "0" ]]; then @@ -217,30 +217,34 @@ function test_multi_app () fi done - wait_for_message "${low_output_file_pattern}" "exit: 0" - ret=$? - ok $ret "Application for low notification terminated normally" - if [[ $ret -eq "0" ]]; then - rm ${CURDIR}/${low_output_file_pattern}* 2> /dev/null + wait_for_message "${tmpdir}" "${low_output_file_pattern}" "exit: 0" + ret_low=$? + ok $ret_low "Application for low notification terminated normally" + if [[ $ret_low -eq "0" ]]; then + rm -rf ${tmpdir}/${low_output_file_pattern}* 2> /dev/null else # Keep the file - print_errors "${low_output_file_pattern}" + print_errors "${tmpdir}" "${low_output_file_pattern}" fi - wait_for_message "${high_output_file_pattern}" "exit: 0" - ret=$? - ok $ret "Application for high notification terminated normally" - if [[ $ret -eq "0" ]]; then - rm ${CURDIR}/${high_output_file_pattern}* 2> /dev/null + wait_for_message "${tmpdir}" "${high_output_file_pattern}" "exit: 0" + ret_high=$? + ok $ret_high "Application for high notification terminated normally" + if [[ $ret_high -eq "0" ]]; then + rm -rf ${tmpdir}/${high_output_file_pattern}* 2> /dev/null else # Keep the file - print_errors "${high_output_file_pattern}" + print_errors "${tmpdir}" "${high_output_file_pattern}" fi + stop_lttng_sessiond kill -9 $app_pid wait $app_pid 2> /dev/null - stop_lttng_sessiond + if [[ $ret_low -eq "0" && $ret_high -eq "0" ]]; then + rm -rf ${tmpdir} + fi + return 0 } function test_on_register_evaluation () @@ -248,11 +252,10 @@ function test_on_register_evaluation () local app_pids=() local high_output_file_pattern="high_app_output_file_on_register_evaluation" - local testpoint_base_path=$(readlink -f "$CURDIR/lttng.t_p_n_register_evaluation") + local tmpdir=$(mktemp -d) + local testpoint_base_path=$(readlink -f "${tmpdir}/lttng.t_p_n_register_evaluation") local testpoint_pipe_path=$(mktemp -u "${testpoint_base_path}.XXXXXX") - - # Cleanup - rm ${CURDIR}/${high_output_file_pattern}* 2> /dev/null + local consumerd_pipe=() # Setup LTTNG_SESSIOND_ENV_VARS="LTTNG_TESTPOINT_ENABLE=1 CONSUMER_PAUSE_PIPE_PATH=${testpoint_pipe_path} LD_PRELOAD=${TESTPOINT}" @@ -276,15 +279,15 @@ function test_on_register_evaluation () # Fetch consumerd testpoint pipe information # This is needed since the testpoint create a pipe with the consumer type suffixed - for f in "$testpoint_base_path"*; do + for f in "$testpoint_pipe_path"*; do consumerd_pipe+=("$f") done high_app_output_file=${high_output_file_pattern}.first_receiver - high_app_output_path=$CURDIR/${high_app_output_file} + high_app_output_path=${tmpdir}/${high_app_output_file} start_client $high_app_output_path $SESSION_NAME $UST_CHANNEL_NAME LTTNG_DOMAIN_UST HIGH RATIO 0.420 1 - wait_for_message "${high_app_output_file}" "sync: ready" + wait_for_message "${tmpdir}" "${high_app_output_file}" "sync: ready" # Stop consumerd consumption to force high notification start_lttng_tracing_ok $SESSION_NAME @@ -293,35 +296,36 @@ function test_on_register_evaluation () stop_consumerd "${pipe}" done - wait_for_message "${high_app_output_file}" "notification: high 0" + wait_for_message "${tmpdir}" "${high_app_output_file}" "notification: high 0" # Start a second receiver, the receiver should receive a high # notification on subscription high_app_output_file=${high_output_file_pattern}.second_receiver - high_app_output_path=$CURDIR/${high_app_output_file} + high_app_output_path=${tmpdir}/${high_app_output_file} start_client $high_app_output_path $SESSION_NAME $UST_CHANNEL_NAME LTTNG_DOMAIN_UST HIGH RATIO 0.420 1 - wait_for_message "${high_app_output_file}" "sync: ready" - wait_for_message "${high_app_output_file}" "notification: high 0" + wait_for_message "${tmpdir}" "${high_app_output_file}" "sync: ready" + wait_for_message "${tmpdir}" "${high_app_output_file}" "notification: high 0" # Resume consumerd for pipe in "${consumerd_pipe[@]}"; do resume_consumerd "${pipe}" done - wait_for_message "${high_output_file_pattern}" "exit: 0" - ret=$? - ok $ret "Application for high notification terminated normally" - if [[ $ret -eq "0" ]]; then - rm ${CURDIR}/${high_output_file_pattern}* 2> /dev/null - else - # Keep the file - print_errors "${high_output_file_pattern}" - fi + wait_for_message "${tmpdir}" "${high_output_file_pattern}" "exit: 0" + ret_high=$? + ok $ret_high "Application for high notification terminated normally" stop_lttng_sessiond - kill -9 $app_pid wait $app_pid 2> /dev/null + + if [[ $ret_high -eq "0" ]]; then + rm -rf ${tmpdir} 2> /dev/null + else + # Keep the file + print_errors "${tmpdir}" "${high_output_file_pattern}" + fi + return 0 } @@ -335,9 +339,6 @@ do TRACE_PATH=$(mktemp -d) ${fct_test} - if [ $? -ne 0 ]; then - break; - fi # Only delete if successful rm -rf $TRACE_PATH -- 2.11.0 _______________________________________________ lttng-dev mailing list lttng-dev@lists.lttng.org https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev