diff options
author | ShwethaHP <spandura@redhat.com> | 2017-08-23 17:24:57 +0530 |
---|---|---|
committer | ShwethaHP <spandura@redhat.com> | 2017-08-28 10:15:34 +0530 |
commit | dfe392e04ee3a126fad2a9b3b5c526f7f7594cf5 (patch) | |
tree | 9eeb1751c01b4cb45e6624a613625273d52ec5af /tests/functional/bvt/test_cvt.py | |
parent | a8701fc8c8855429e9fc7da99dbe2b6b952802f2 (diff) |
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 <spandura@redhat.com>
Diffstat (limited to 'tests/functional/bvt/test_cvt.py')
-rw-r--r-- | tests/functional/bvt/test_cvt.py | 25 |
1 files changed, 17 insertions, 8 deletions
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) |