From: Hongchao Zhang <hongchao.zhang@intel.com>
Date: Mon, 27 Jun 2016 12:24:56 +0000 (+0800)
Subject: LU-8333 test: make sure COS is cleared
X-Git-Tag: 2.8.58~39
X-Git-Url: https://git.whamcloud.com/?p=fs%2Flustre-release.git;a=commitdiff_plain;h=ae2ccbc6da32def014463ee98fab76a93835d85d

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 <hongchao.zhang@intel.com>
Reviewed-on: http://review.whamcloud.com/21924
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
---

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 ] &&