From ae2ccbc6da32def014463ee98fab76a93835d85d Mon Sep 17 00:00:00 2001 From: Hongchao Zhang Date: Mon, 27 Jun 2016 20:24:56 +0800 Subject: [PATCH] LU-8333 test: make sure COS is cleared In subtest 21b of replay-dual, the COS could be set after the MDT is failed over, and the test will fail in this case Change-Id: I9401b905593c76f8fddfab19ab9eb6c0fe886e41 Signed-off-by: Hongchao Zhang Reviewed-on: http://review.whamcloud.com/21924 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Lai Siyao Reviewed-by: Fan Yong Reviewed-by: Oleg Drokin --- lustre/tests/replay-dual.sh | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lustre/tests/replay-dual.sh b/lustre/tests/replay-dual.sh index 8b63d16..65d3106 100755 --- a/lustre/tests/replay-dual.sh +++ b/lustre/tests/replay-dual.sh @@ -578,15 +578,15 @@ test_21b() { test_21b_sub $facet || error "Not all renames are replayed. COS=$COS" - # COS disabled (should fail) - COS=0 - do_facet $facet lctl set_param mdt.*.commit_on_sharing=$COS - # there is still a window when transactions may be written to disk # before the mds device is set R/O. To avoid such a rare test failure, # the check is repeated several times. + COS=0 local n_attempts=1 while true; do + # COS disabled (should fail) + do_facet $facet lctl set_param mdt.*.commit_on_sharing=$COS + test_21b_sub $facet || break n_attempts=$((n_attempts + 1)) [ $n_attempts -gt 3 ] && -- 1.8.3.1