Whamcloud - gitweb
LU-16838 tests: use import name in 398a 64/51064/3
authorPatrick Farrell <pfarrell@whamcloud.com>
Fri, 19 May 2023 16:24:25 +0000 (12:24 -0400)
committerOleg Drokin <green@whamcloud.com>
Wed, 13 Sep 2023 04:02:54 +0000 (04:02 +0000)
The LU-15670 test change assumes ost1_import is always
OST0000.  This isn't quite always true, so the test is
failing in certain configurations.

Change it to use the import name.

Fixes: 649d638467 ("LU-15670 clio: Disable lockless for DIO with O_APPEND")
Test-parameters: trivial
Signed-off-by: Patrick Farrell <pfarrell@whamcloud.com>
Change-Id: Ifaefc503d1118ecd6fd45b661cbe94607f7ad799
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/51064
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Shaun Tancheff <shaun.tancheff@hpe.com>
lustre/tests/sanity.sh

index 099f16e..e4eab11 100755 (executable)
@@ -26357,7 +26357,7 @@ test_398a() { # LU-4198
                           ldlm.namespaces.$imp_name.lru_size)
        [[ $lock_count -eq 0 ]] || error "lock should be cancelled by direct IO"
 
-       $LCTL set_param ldlm.namespaces.*-OST0000-osc-ffff*.lru_size=clear
+       $LCTL set_param ldlm.namespaces.$imp_name.lru_size=clear
 
        # no lock cached, should use lockless DIO and not enqueue new lock
        dd if=/dev/zero of=$DIR/$tfile bs=1M count=1 oflag=direct conv=notrunc
@@ -26365,13 +26365,13 @@ test_398a() { # LU-4198
                     ldlm.namespaces.$imp_name.lru_size)
        [[ $lock_count -eq 0 ]] || error "no lock should be held by direct IO"
 
-       $LCTL set_param ldlm.namespaces.*-OST0000-osc-ffff*.lru_size=clear
+       $LCTL set_param ldlm.namespaces.$imp_name.lru_size=clear
 
        # no lock cached, should use locked DIO append
        dd if=/dev/zero of=$DIR/$tfile bs=1M count=1 oflag=direct oflag=append \
                conv=notrunc || error "DIO append failed"
        lock_count=$($LCTL get_param -n \
-                    ldlm.namespaces.*-OST0000-osc-ffff*.lru_size)
+                    ldlm.namespaces.$imp_name.lru_size)
        [[ $lock_count -ne 0 ]] || error "lock still must be held by DIO append"
 }
 run_test 398a "direct IO should cancel lock otherwise lockless"