Commit d8b8bc43 authored by Jim Harris's avatar Jim Harris
Browse files

test/lvol: wait for lvolstore tasting after nvme bdev construction 



A couple of the lvol tests delete the nvme bdev, then construct
it again to test lvolstore tasting.  But constructing the nvme
bdev and then the tasting can take a tiny bit of time, and if
we check for the lvolstore too soon, it could fail.  There are
a large number of intermittent test failures in the test pool
that show this signature.

So sleep one second to allow time for the tasting to complete.

Signed-off-by: default avatarJim Harris <james.r.harris@intel.com>
Change-Id: Iebabe09458302d8300e704f232fdbb245b562268

Reviewed-on: https://review.gerrithub.io/392559


Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarKarol Latecki <karol.latecki@intel.com>
parent 1545c8eb

test/lvol/test_cases.py

+4 −0
Original line number Diff line number Diff line
@@ -527,6 +527,8 @@ class TestCases(object): 
        if traddr != -1:

            self.c.delete_bdev(base_name)

            self.c.construct_nvme_bdev("Nvme0", "PCIe", traddr)

            # wait 1 second to allow time for lvolstore tasting

            sleep(1)

        else:

            fail_count += 1

        ret_value = self.c.check_get_lvol_stores(base_name, uuid_store,
@@ -791,6 +793,8 @@ class TestCases(object): 
        if traddr != -1:

            self.c.delete_bdev(base_name)

            self.c.construct_nvme_bdev("Nvme0", "PCIe", traddr)

            # wait 1 second to allow time for lvolstore tasting

            sleep(1)

        else:

            fail_count += 1

        if self.c.check_get_lvol_stores(base_name, uuid_store,