From 1fcfd6e62e80673ecadda43e22892867bd60fd02 Mon Sep 17 00:00:00 2001 From: Jian Yu Date: Fri, 19 Apr 2024 11:54:04 -0700 Subject: [PATCH] LU-17761 tests: make sanity-compr sanity/sanityn return 0 While running sanity-compr sanity/sanityn, if there was sub-subtest failure, the sanity/sanityn test_cleanup would be incorrectly marked as FAIL. We should leave it to the individual sanity/sanityn subtests to mark their failures, test_sanity() and test_sanityn() should not also return an error. Change-Id: I1fd645b80b92e583f1a564f85e6d2d6d871b8fa8 Test-Parameters: trivial testlist=sanity-compr Signed-off-by: Jian Yu Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/54855 Reviewed-by: Andreas Dilger Reviewed-by: Artem Blagodarenko Reviewed-by: Oleg Drokin Tested-by: jenkins Tested-by: Maloo --- lustre/tests/sanity-compr.sh | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/lustre/tests/sanity-compr.sh b/lustre/tests/sanity-compr.sh index a211636..135ceba 100644 --- a/lustre/tests/sanity-compr.sh +++ b/lustre/tests/sanity-compr.sh @@ -36,21 +36,16 @@ $LFS setstripe $COMPR_EXTRA_LAYOUT $MOUNT test_sanity() { always_except LU-16928 56wb - local rc=0 SANITY_EXCEPT=$ALWAYS_EXCEPT bash sanity.sh - rc=$? - return $rc + return 0 } run_test sanity "Run sanity with PFL layout" test_sanityn() { - local rc=0 - bash sanityn.sh - rc=$? - return $rc + return 0 } run_test sanityn "Run sanityn with PFL layout" -- 1.8.3.1