From d8b8bc4353b7c551c0b722186129c6f64369cdb5 Mon Sep 17 00:00:00 2001 From: Jim Harris Date: Wed, 20 Dec 2017 13:07:39 -0700 Subject: [PATCH] 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: Jim Harris Change-Id: Iebabe09458302d8300e704f232fdbb245b562268 Reviewed-on: https://review.gerrithub.io/392559 Reviewed-by: Daniel Verkamp Tested-by: SPDK Automated Test System Reviewed-by: Ben Walker Reviewed-by: Karol Latecki --- test/lvol/test_cases.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/test/lvol/test_cases.py b/test/lvol/test_cases.py index 0a9be4f80..8bbdd21f4 100644 --- a/test/lvol/test_cases.py +++ b/test/lvol/test_cases.py @@ -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,