Whamcloud - gitweb
LU-16280 tests: conf-sanity/117 call setup when FS is not mounted 89/48989/3
authorArshad Hussain <arshad.hussain@aeoncomputing.com>
Mon, 31 Oct 2022 14:01:32 +0000 (19:31 +0530)
committerOleg Drokin <green@whamcloud.com>
Tue, 8 Nov 2022 08:53:20 +0000 (08:53 +0000)
conf-sanity/117 calls setup() funciton to setup LustreFS
regardless of the previous state of conf-sanity. Currently
test before conf-sanity/117 leaves state of conf-sanity
as not mounted. However, this make test case highly
depenending on previous state. Specially if tests are
unrelated.

This patch checks the state of LustreFS before calling
setup.

Test-Parameters: trivial fstype=zfs testlist=conf-sanity env=ONLY="112 115 116 117"
Signed-off-by: Arshad Hussain <arshad.hussain@aeoncomputing.com>
Change-Id: I40c2b91f9cf5138f0f3c377fd222d092b37d1c12
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/48989
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Jian Yu <yujian@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/tests/conf-sanity.sh

index 5fd625e..b05cbc6 100644 (file)
@@ -9210,7 +9210,9 @@ test_116() {
 run_test 116 "big size MDT support"
 
 test_117() {
-       setup
+       # Call setup only if LustreFS is not mounted
+       check_mount || setup
+
        do_facet ost1 "$LCTL set_param ost.OSS.ost_io.nrs_policies=fifo"
        do_facet ost1 "$LCTL get_param -n ost.OSS.ost_io.nrs_tbf_rule" &&
                error "get_param should fail"