From 29d4d4095a98fb5272112c6128f017dd228d96c2 Mon Sep 17 00:00:00 2001 From: Mr NeilBrown Date: Thu, 13 Aug 2020 10:26:40 +1000 Subject: [PATCH] LU-4671 tests: give multiop a chance to exit. If 'multiop' is still running after a test complete, test-framework.sh reports a failure. test_43A signals multiop asking it to exit as the last thing it does. If there is any delay in multip being schedule, test-framework will see it and report an error - this is a false negative. So use 'wait' to wait for multiop to respond to the signal. Test-Parameters: trivial testlist=sanity envdefinitions=ONLY="43A" Signed-off-by: Mr NeilBrown Change-Id: Ic94d3c89bd98c9f6ef2d5bee22aac1e39116bf11 Reviewed-on: https://review.whamcloud.com/39665 Tested-by: jenkins Reviewed-by: Andreas Dilger Reviewed-by: Jian Yu Reviewed-by: James Nunez Tested-by: Maloo --- lustre/tests/sanity.sh | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lustre/tests/sanity.sh b/lustre/tests/sanity.sh index 335c536..d69da54 100755 --- a/lustre/tests/sanity.sh +++ b/lustre/tests/sanity.sh @@ -4880,6 +4880,8 @@ test_43A() { # was test_43 $DIR/$tdir/$tfile && error "execute $DIR/$tdir/$tfile succeeded" || true kill -USR1 $pid + # Wait for multiop to exit + wait $pid } run_test 43A "execution of file opened for write should return -ETXTBSY" -- 1.8.3.1