rgw: remove unnecessary sleeps

remove the sleeps that were added to address radosgw startup races

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 3e68bfdbb29e933edb06f73c88eed176ffacc2e3)

diff --git a/suites/rgw/singleton/all/radosgw-admin-data-sync.yaml b/suites/rgw/singleton/all/radosgw-admin-data-sync.yaml
index 33322b1dafac484f82d3f2b9afedd3962e232a19..328641ea0a124f042aa86e2a45461ec47c9b12da 100644 (file)
--- a/suites/rgw/singleton/all/radosgw-admin-data-sync.yaml
+++ b/suites/rgw/singleton/all/radosgw-admin-data-sync.yaml
@@ -50,8 +50,6 @@ tasks:
         name: client1-system-user
         access key: 1te6NH5mcdcq0Tc5i8i3
         secret key: Py4IOauQoL18Gp2zM7lC1vLmoawgqcYPbYGcWfXw
-- sleep:
-    duration: 15
 - radosgw-agent:
     client.0:
       max-entries: 10

diff --git a/suites/rgw/singleton/all/radosgw-admin-multi-region.yaml b/suites/rgw/singleton/all/radosgw-admin-multi-region.yaml
index 10dc1d80553edfa52f74c564abe9b027a7d67634..8e1c7194e01c448ff0e215aea03c58142555f7bd 100644 (file)
--- a/suites/rgw/singleton/all/radosgw-admin-multi-region.yaml
+++ b/suites/rgw/singleton/all/radosgw-admin-multi-region.yaml
@@ -55,8 +55,6 @@ tasks:
         name: client1-system-user
         access key: 1te6NH5mcdcq0Tc5i8i3
         secret key: Py4IOauQoL18Gp2zM7lC1vLmoawgqcYPbYGcWfXw
-- sleep:
-    duration: 5
 - radosgw-agent:
     client.0:
       src: client.0

diff --git a/suites/rgw/verify/tasks/rgw_s3tests_multiregion.yaml b/suites/rgw/verify/tasks/rgw_s3tests_multiregion.yaml
index fba726c7d2190e709824e607e99404f365422815..611a0ddcb3adbdd61d4634af1047ca1d4e3f38f9 100644 (file)
--- a/suites/rgw/verify/tasks/rgw_s3tests_multiregion.yaml
+++ b/suites/rgw/verify/tasks/rgw_s3tests_multiregion.yaml
@@ -51,8 +51,6 @@ tasks:
         name: client1-system-user
         access key: 0te6NH5mcdcq0Tc5i8i2
         secret key: Oy4IOauQoL18Gp2zM7lC1vLmoawgqcYPbYGcWfXv
-- sleep:
-    duration: 5
 - radosgw-agent:
     client.0:
       src: client.0

diff --git a/tasks/rgw.py b/tasks/rgw.py
index c1194e6116f1017280f87d4c7ea71c25ddd794ad..b39fa10ea6b8f3992944dba2d851fcb896fcb6f5 100644 (file)
--- a/tasks/rgw.py
+++ b/tasks/rgw.py
@@ -1032,12 +1032,6 @@ def pull_configuration(ctx, config, regions, role_endpoints, realm, master_clien
 
     yield
 
-@contextlib.contextmanager
-def wait_for_master():
-    log.debug("wait_for_master")
-    time.sleep(20)
-    yield
-
 @contextlib.contextmanager
 def task(ctx, config):
     """
@@ -1296,7 +1290,6 @@ def task(ctx, config):
         else:
             raise ValueError("frontend must be 'apache' or 'civetweb'")
 
-        subtasks.extend([lambda: wait_for_master(),])
         subtasks.extend([
             lambda: pull_configuration(ctx=ctx,
                                        config=config,