From dfe392e04ee3a126fad2a9b3b5c526f7f7594cf5 Mon Sep 17 00:00:00 2001 From: ShwethaHP Date: Wed, 23 Aug 2017 17:24:57 +0530 Subject: 1) bring_bricks_online: Wait for bricks to be online for 30 seconds after bringing them online. 2) log all the xml output/error to DEBUG log level. Change-Id: If6bb758ac728f299292def9d72c0ef166a1569ae Signed-off-by: ShwethaHP --- tests/functional/bvt/test_cvt.py | 25 +++++++++++++++++-------- 1 file changed, 17 insertions(+), 8 deletions(-) (limited to 'tests/functional/bvt') diff --git a/tests/functional/bvt/test_cvt.py b/tests/functional/bvt/test_cvt.py index b367004ea..e19adf862 100644 --- a/tests/functional/bvt/test_cvt.py +++ b/tests/functional/bvt/test_cvt.py @@ -684,14 +684,6 @@ class TestGlusterHealSanity(GlusterBasicFeaturesSanityBaseClass): # Wait for gluster processes to be offline time.sleep(10) - # Validate if bricks are offline - g.log.info("Validating if bricks: %s are offline", - bricks_to_bring_offline) - ret = are_bricks_offline(self.mnode, self.volname, - bricks_to_bring_offline) - self.assertTrue(ret, "Not all the bricks in list:%s are offline") - g.log.info("Successfully validated that bricks: %s are all offline") - # Log Volume Info and Status g.log.info("Logging volume info and Status after bringing bricks " "offline from the volume %s", self.volname) @@ -701,6 +693,14 @@ class TestGlusterHealSanity(GlusterBasicFeaturesSanityBaseClass): g.log.info("Successful in logging volume info and status of volume %s", self.volname) + # Validate if bricks are offline + g.log.info("Validating if bricks: %s are offline", + bricks_to_bring_offline) + ret = are_bricks_offline(self.mnode, self.volname, + bricks_to_bring_offline) + self.assertTrue(ret, "Not all the bricks in list:%s are offline") + g.log.info("Successfully validated that bricks: %s are all offline") + # Add delay before bringing bricks online time.sleep(40) @@ -716,6 +716,15 @@ class TestGlusterHealSanity(GlusterBasicFeaturesSanityBaseClass): # Wait for gluster processes to be online time.sleep(10) + # Log Volume Info and Status + g.log.info("Logging volume info and Status after bringing bricks " + "online from the volume %s", self.volname) + ret = log_volume_info_and_status(self.mnode, self.volname) + self.assertTrue(ret, ("Logging volume info and status failed on " + "volume %s", self.volname)) + g.log.info("Successful in logging volume info and status of volume %s", + self.volname) + # Verify volume's all process are online g.log.info("Verifying volume's all process are online") ret = verify_all_process_of_volume_are_online(self.mnode, self.volname) -- cgit