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 <pfarrell@whamcloud.com>
Change-Id: I3046303fcc4e10364de9f673fab2142c8cecff64
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/51183
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>

diff --git a/lustre/tests/sanity.sh b/lustre/tests/sanity.sh
index 12a086f..66f3dc0 100755 (executable)
--- 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'"