From b9a667914fb60b9f37a4bd58de5b48b58b423a34 Mon Sep 17 00:00:00 2001 From: Alexis Lescouet Date: Fri, 11 Feb 2022 17:32:59 +0000 Subject: [PATCH] test/common: Remove the call to sudo in unit test script when not needed Signed-off-by: Alexis Lescouet Change-Id: I457ddee7c110088578b7f1b0a5caf78882e301c5 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/11636 Tested-by: SPDK CI Jenkins Community-CI: Broadcom CI Reviewed-by: Changpeng Liu Reviewed-by: Dong Yi Reviewed-by: Ben Walker Reviewed-by: Aleksey Marchuk --- test/common/autotest_common.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/common/autotest_common.sh b/test/common/autotest_common.sh index 4792818eb..2c3dfa00e 100755 --- a/test/common/autotest_common.sh +++ b/test/common/autotest_common.sh @@ -183,7 +183,7 @@ export UBSAN_OPTIONS='halt_on_error=1:print_stacktrace=1:abort_on_error=1:disabl # Export LeakSanitizer option to use suppression file in order to prevent false positives # and known leaks in external executables or libraries from showing up. asan_suppression_file="/var/tmp/asan_suppression_file" -sudo rm -rf "$asan_suppression_file" +rm -rf "$asan_suppression_file" 2> /dev/null || sudo rm -rf "$asan_suppression_file" cat << EOL >> "$asan_suppression_file" # ASAN has some bugs around thread_local variables. We have a destructor in place # to free the thread contexts, but ASAN complains about the leak before those