From a4d0b9612568d20baec2b53a25119bf1e697234e Mon Sep 17 00:00:00 2001 From: Elena Gryaznova Date: Tue, 19 Oct 2021 21:10:31 +0300 Subject: [PATCH] LU-15131 tests: check parameter correctly sanity test_254() always skipped due to wrong parameter path: MDT0 not initialized before get_param. Fixes: 33aad7829de5 ("LU-10461 tests: call exit in the skip routine") Test-Parameters: trivial testlist=sanity Signed-off-by: Elena Gryaznova HPE-bug-id: LUS-9969 Reviewed-by: Alexander Boyko Reviewed-by: Alexander Zarochentsev Change-Id: I02e38a27ca4656128e62339d63425f85386fa905 Reviewed-on: https://review.whamcloud.com/45300 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Oleg Drokin --- lustre/tests/sanity.sh | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/lustre/tests/sanity.sh b/lustre/tests/sanity.sh index 39ab34d..c352d89 100755 --- a/lustre/tests/sanity.sh +++ b/lustre/tests/sanity.sh @@ -21196,18 +21196,20 @@ run_test 253 "Check object allocation limit" test_254() { [ $PARALLEL == "yes" ] && skip "skip parallel run" remote_mds_nodsh && skip "remote MDS with nodsh" - do_facet $SINGLEMDS $LCTL get_param -n mdd.$MDT0.changelog_size || + + local mdt=$(facet_svc $SINGLEMDS) + + do_facet $SINGLEMDS $LCTL get_param -n mdd.$mdt.changelog_size || skip "MDS does not support changelog_size" local cl_user - local MDT0=$(facet_svc $SINGLEMDS) changelog_register || error "changelog_register failed" changelog_clear 0 || error "changelog_clear failed" local size1=$(do_facet $SINGLEMDS \ - $LCTL get_param -n mdd.$MDT0.changelog_size) + $LCTL get_param -n mdd.$mdt.changelog_size) echo "Changelog size $size1" rm -rf $DIR/$tdir @@ -21222,7 +21224,7 @@ test_254() { rm $DIR/$tdir/pics/desktop.jpg local size2=$(do_facet $SINGLEMDS \ - $LCTL get_param -n mdd.$MDT0.changelog_size) + $LCTL get_param -n mdd.$mdt.changelog_size) echo "Changelog size after work $size2" (( $size2 > $size1 )) || -- 1.8.3.1