Whamcloud - gitweb
LU-15445 tests: sanity test_160p() fix 73/46073/2
authorElena Gryaznova <elena.gryaznova@hpe.com>
Wed, 12 Jan 2022 17:03:02 +0000 (20:03 +0300)
committerOleg Drokin <green@whamcloud.com>
Wed, 26 Jan 2022 05:16:44 +0000 (05:16 +0000)
start() requires 2nd parameter device.
If start() is called without the 2nd parameter - the
empty mds1_dev is exported:
        eval export ${dev_alias}_dev=${device}
and test fails on failover setup with:
  CMD: lm0301 loop_dev=$(losetup -j  | cut -d : -f 1);
  lm0301: losetup: option requires an argument -- 'j'
dm_create_dev()
  local real_dev=<empty>
        -> setup_loop_device $facet <empty>
To reproduce the failure just run:
  ONLY=160p sh sanity.sh
on failover setup where mds1_HOST != mds1failover_HOST.

Fixes: c7d8fe3106 ("LU-14731 mdd: clear orphans changelog entries")
Test-Parameters: trivial env=ONLY="160p" testlist=sanity
Signed-off-by: Elena Gryaznova <elena.gryaznova@hpe.com>
HPE-bug-id: LUS-10674
Reviewed-by: Vladimir Saveliev <vlaidimir.saveliev@hpe.com>
Reviewed-by: Alexander Boyko <alexander.boyko@hpe.com>
Change-Id: I2661567672aa9c6e23b5f17500d81053cf9c9fdd
Reviewed-on: https://review.whamcloud.com/46073
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/tests/sanity.sh

index c9df3fd..fdc9d84 100755 (executable)
@@ -16824,8 +16824,9 @@ test_160p() {
 
        # remove changelog_users and check that orphan entries are removed
        stop mds1
-       do_facet mds1 "$DEBUGFS -w -R 'rm changelog_users' $(mdsdevname 1)"
-       start mds1 || error "cannot start mdt"
+       local dev=$(mdsdevname 1)
+       do_facet mds1 "$DEBUGFS -w -R 'rm changelog_users' $dev"
+       start mds1 $dev $MDS_MOUNT_OPTS || error "cannot start mds1"
        entry_count=$(changelog_dump | wc -l)
        ((entry_count == 0)) ||
                error "found $entry_count changelog entries, expected none"