From 4ac0324fb9d824915b3dd11b75e81e609d9e8e84 Mon Sep 17 00:00:00 2001 From: Patrick Farrell Date: Tue, 28 May 2019 17:02:49 -0400 Subject: [PATCH 1/1] LU-12350 tests: Do not use background failover For some reason, test 33 chooses at one point to take an OST offline by starting failover in the background. It seems to assume the OST will be offline during the subsequent read, without doing anything to verify it is offline - In fact, it could either be not offline yet or back online with failover complete. Just use stop like the rest of the test does. Signed-off-by: Patrick Farrell Change-Id: I9c074ff1412793b8f0d8f15dc1e2ee21bb6d9fd6 Reviewed-on: https://review.whamcloud.com/34985 Reviewed-by: Andreas Dilger Tested-by: Jenkins Reviewed-by: James Nunez Tested-by: Maloo Reviewed-by: Oleg Drokin --- lustre/tests/sanity-flr.sh | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/lustre/tests/sanity-flr.sh b/lustre/tests/sanity-flr.sh index fb94b57b..33732e3 100644 --- a/lustre/tests/sanity-flr.sh +++ b/lustre/tests/sanity-flr.sh @@ -1064,10 +1064,8 @@ test_33() { start_osts 1 # read file again with ost2 failed - $LCTL set_param ldlm.namespaces.lustre-*-osc-[-0-9a-f]*.lru_size=clear - - fail ost2 & - sleep 1 + stop_osts 2 + drop_client_cache # check size, glimpse should work $CHECKSTAT -t file -s $fsize $DIR/$tfile || @@ -1078,7 +1076,7 @@ test_33() { [[ "$rs" == "ost1" ]] || error "file content error: expected: \"ost1\", actual: \"$rs\"" - wait_osc_import_ready client ost2 + start_osts 2 } run_test 33 "read can choose available mirror to read" -- 1.8.3.1