diff options
| author | Sri Vignesh <sselvan@redhat.com> | 2020-02-24 13:46:27 +0530 |
|---|---|---|
| committer | Sri Vignesh <sselvan@redhat.com> | 2020-02-25 09:27:02 +0000 |
| commit | 87638f2ab5379c04dfdf87329ebe450c24ac25bc (patch) | |
| tree | 4456f11d1dfba028bdee646d53490e4938afbf06 /tests/functional/glusterd/test_remove_brick_after_restart_glusterd.py | |
| parent | 16a3b1f8dd1492fe45861cbd9f3a7d10f988c843 (diff) | |
[testfix] Add steps to stabilise glusterd
Moved steps from teardown class to teardown and removed
unwanted teardown class and rectified the testcase
failing with wait for io to complete by removing the step
because after validate io the sub process terminates and
results in failure.
Change-Id: I2eaf05680b817b681aff8b48683fc9dac88896b0
Signed-off-by: Sri Vignesh <sselvan@redhat.com>
Diffstat (limited to 'tests/functional/glusterd/test_remove_brick_after_restart_glusterd.py')
| -rw-r--r-- | tests/functional/glusterd/test_remove_brick_after_restart_glusterd.py | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/tests/functional/glusterd/test_remove_brick_after_restart_glusterd.py b/tests/functional/glusterd/test_remove_brick_after_restart_glusterd.py index 90e962b39..40012b009 100644 --- a/tests/functional/glusterd/test_remove_brick_after_restart_glusterd.py +++ b/tests/functional/glusterd/test_remove_brick_after_restart_glusterd.py @@ -29,7 +29,7 @@ from glustolibs.gluster.lib_utils import form_bricks_list from glustolibs.gluster.brick_ops import remove_brick from glustolibs.gluster.exceptions import ExecutionError from glustolibs.gluster.mount_ops import mount_volume, umount_volume -from glustolibs.io.utils import validate_io_procs, wait_for_io_to_complete +from glustolibs.io.utils import validate_io_procs from glustolibs.gluster.gluster_init import restart_glusterd @@ -166,11 +166,9 @@ class TestRemoveBrickAfterRestartGlusterd(GlusterBaseClass): self.io_validation_complete = False # Validate IO - ret = validate_io_procs(self.all_mounts_procs, self.mounts) - self.assertTrue(ret, "IO failed on some of the clients") - if ret: - wait_for_io_to_complete(self.all_mounts_procs, self.mounts) - g.log.info("wait for io completed") + self.assertTrue( + validate_io_procs(self.all_mounts_procs, self.mounts), + "IO failed on some of the clients") self.io_validation_complete = True remove_brick_list = bricks_list[2:4] |