From 81f309421dafa9a7519634bb27f54faabd5ef852 Mon Sep 17 00:00:00 2001 From: Emoly Liu Date: Mon, 19 Sep 2016 14:13:16 +0800 Subject: [PATCH] LU-8021 tests: Add leading $ to "DEBUGSAVE_SERVER" Add leading $ to "DEBUGSAVE_SERVER", otherwise the check is always true and the code tries to restore an empty string and it produces a lot of spurious error messages in the test logs. Test-Parameters: trivial Signed-off-by: Emoly Liu Change-Id: I4e9206bc312af22fea22bf6ab634469439008003 Reviewed-on: http://review.whamcloud.com/22586 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Andreas Dilger Reviewed-by: James Nunez Reviewed-by: Parinay Kondekar --- lustre/tests/test-framework.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lustre/tests/test-framework.sh b/lustre/tests/test-framework.sh index 95ddf56..a9097c1 100755 --- a/lustre/tests/test-framework.sh +++ b/lustre/tests/test-framework.sh @@ -4731,7 +4731,7 @@ debugrestore() { true DEBUGSAVE="" - [ -n "DEBUGSAVE_SERVER" ] && + [ -n "$DEBUGSAVE_SERVER" ] && do_nodes $(comma_list $(all_server_nodes)) \ "$LCTL set_param debug=\\\"${DEBUGSAVE_SERVER}\\\"" || true -- 1.8.3.1