From 1dbe9be20011893ca46ccbbd2676e8063af4158d Mon Sep 17 00:00:00 2001 From: Hongchao Zhang Date: Thu, 21 Sep 2023 23:22:17 -0400 Subject: [PATCH] EX-7342 test: remove extra cleanup and qp check The test_79 in sanity-quota needs quota pool support, and the cleanup of the "stop file" is also included in the stack_trap, then it is no need to to do it explicitly. Test-Parameters: trivial Signed-off-by: Hongchao Zhang Change-Id: If86a1d0187b4b95d0c5e24f11f5f058280726e64 Reviewed-on: https://review.whamcloud.com/c/ex/lustre-release/+/52472 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Andreas Dilger --- lustre/tests/sanity-quota.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lustre/tests/sanity-quota.sh b/lustre/tests/sanity-quota.sh index 075c43d..e3d9489 100755 --- a/lustre/tests/sanity-quota.sh +++ b/lustre/tests/sanity-quota.sh @@ -5533,12 +5533,12 @@ test_79() local cmd="$LCTL get_param -n qmt.$FSNAME-QMT0000.dt-$qpool.info" local stopf=$TMP/$tfile + mds_supports_qp do_facet mds1 "touch $stopf" stack_trap "do_facet mds1 'rm -f $stopf'" do_facet mds1 "while [ -e $stopf ]; do $cmd &>/dev/null; done"& local pid=$! pool_add $qpool || error "pool_add failed" - do_facet mds1 "rm $stopf" wait $pid } run_test 79 "access to non-existed dt-pool/info doesn't cause a panic" -- 1.8.3.1