From 347130583137ed9eb779b61d3afbb697e0bc2772 Mon Sep 17 00:00:00 2001 From: Andreas Dilger Date: Tue, 18 Dec 2018 11:57:46 -0700 Subject: [PATCH] LU-11797 tests: improve sanity test 272a checking Improve sanity.sh test_272a() to check data integrity before it checks the stripe count. Also, print out the stripe count if it does not match our expectations. Test-Parameters: trivial Signed-off-by: Andreas Dilger Change-Id: I75397f251f10ed00c3dea9f80243d7ba9eacab07 Reviewed-on: https://review.whamcloud.com/33889 Reviewed-by: James Nunez Reviewed-by: Mike Pershin Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Oleg Drokin --- lustre/tests/sanity.sh | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/lustre/tests/sanity.sh b/lustre/tests/sanity.sh index 4ae7a1d..0e40777 100755 --- a/lustre/tests/sanity.sh +++ b/lustre/tests/sanity.sh @@ -17506,14 +17506,13 @@ test_272a() { $LFS migrate -E 256K -L mdt -E -1 -c2 $dom || error "failed to migrate to the same DoM component" - [ $($LFS getstripe -c $dom) -eq 2 ] || - error "layout was not changed silently" - local new_md5=$(md5sum $dom) - [ "$old_md5" != "$new_md5" ] && + [ "$old_md5" == "$new_md5" ] || error "md5sum differ: $old_md5, $new_md5" - return 0 + + [ $($LFS getstripe -c $dom) -eq 2 ] || + error "migrate stripe count bad: $(LFS getstripe -c $dom) != 2" } run_test 272a "DoM migration: new layout with the same DOM component" -- 1.8.3.1