From 41761f7534caa532784a7947e5ed1b1b74cb5e70 Mon Sep 17 00:00:00 2001 From: Patrick Farrell Date: Wed, 31 May 2023 12:28:14 -0400 Subject: [PATCH] LU-15458 tests: double delay for sync/async test This test verifies that an async request takes less than half of the 'delay' time, but occasionally the async request hits a delay on the file open due to some VM issue on the MDT. This causes a spurious failure. This is already pretty rare, so the simplest thing is to double the delay time, which should make it extremely rare. Setting trivial so it just runs sanity. test-parameters: trivial Signed-off-by: Patrick Farrell Change-Id: I3046303fcc4e10364de9f673fab2142c8cecff64 Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/51183 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Sebastien Buisson Reviewed-by: Andreas Dilger Reviewed-by: Oleg Drokin --- lustre/tests/sanity.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lustre/tests/sanity.sh b/lustre/tests/sanity.sh index 12a086f..66f3dc0 100755 --- a/lustre/tests/sanity.sh +++ b/lustre/tests/sanity.sh @@ -23058,10 +23058,12 @@ test_255a() { error "Ladvise succeeded with conflicting -l and -e arguments" echo "Synchronous ladvise should wait" - local delay=4 + local delay=8 #define OBD_FAIL_OST_LADVISE_PAUSE 0x237 do_nodes $(comma_list $(osts_nodes)) \ $LCTL set_param fail_val=$delay fail_loc=0x237 + stack_trap "do_nodes $(comma_list $(osts_nodes)) \ + $LCTL set_param fail_loc=0" local start_ts=$SECONDS lfs ladvise -a willread $DIR/$tfile || @@ -23084,7 +23086,6 @@ test_255a() { error "Asynchronous advice blocked" fi - do_nodes $(comma_list $(osts_nodes)) $LCTL set_param fail_loc=0 ladvise_willread_performance } run_test 255a "check 'lfs ladvise -a willread'" -- 1.8.3.1