From 35b9575fd02f485f48793bedd5de270b826a82b1 Mon Sep 17 00:00:00 2001 From: Andreas Dilger Date: Tue, 23 Oct 2018 07:19:57 +0800 Subject: [PATCH] LU-11558 tests: re-enable conf-sanity t32_verify_quota() Since patch https://review.whamcloud.com/28020 "LU-3285 test: add Data-on-MDT tests and fixes" landed the call to t32_verify_quota() from conf-sanity.sh t32_test() has been removed, and (I'd guess) that we are no longer verifying that quota is still working correctly after an upgrade. It is unclear why this check was removed. Return it back to the code so that we resume testing that quota is working after an upgrade from an old disk format. Test-Parameters: trivial testlist=conf-sanity Test-Parameters: fstype=zfs testlist=conf-sanity Signed-off-by: Andreas Dilger Change-Id: Ia6bab4907e7cb4e6e5581a6a072cc51ab53ebbe5 Reviewed-on: https://review.whamcloud.com/33423 Reviewed-by: Mike Pershin Tested-by: jenkins Tested-by: Maloo Reviewed-by: Li Dongyang Reviewed-by: Oleg Drokin --- lustre/tests/conf-sanity.sh | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/lustre/tests/conf-sanity.sh b/lustre/tests/conf-sanity.sh index a95399f..f6c0457 100644 --- a/lustre/tests/conf-sanity.sh +++ b/lustre/tests/conf-sanity.sh @@ -2100,11 +2100,10 @@ t32_test() { shall_cleanup_lustre=true $r $LCTL set_param debug="$PTLDEBUG" - # Leave re-enabling this to a separate patch for LU-11558 - # t32_verify_quota $SINGLEMDS $fsname $tmp/mnt/lustre || { - # error_noexit "verify quota failed" - # return 1 - #} + t32_verify_quota $SINGLEMDS $fsname $tmp/mnt/lustre || { + error_noexit "verify quota failed" + return 1 + } if $r test -f $tmp/list; then # -- 1.8.3.1