Skip to content

Commit

Permalink
Increase default wait timeout to 600sec (#153)
Browse files Browse the repository at this point in the history
  • Loading branch information
bsquizz authored Nov 15, 2021
1 parent aa46dfc commit e4da6df
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions bonfire/bonfire.py
Original file line number Diff line number Diff line change
Expand Up @@ -220,8 +220,8 @@ def _prepare_namespace(namespace):
"-t",
required=True,
type=int,
default=300,
help="timeout in sec (default = 300) to wait for resources to be ready",
default=600,
help="timeout in sec (default = 600) to wait for resources to be ready",
)
]

Expand Down
8 changes: 4 additions & 4 deletions bonfire/openshift.py
Original file line number Diff line number Diff line change
Expand Up @@ -477,12 +477,12 @@ def _observe(self, item):
self._update_observed_resources(item)


def wait_for_ready(namespace, restype, name, timeout=300):
def wait_for_ready(namespace, restype, name, timeout=600):
waiter = ResourceWaiter(namespace, restype, name)
return waiter.wait_for_ready(timeout)


def wait_for_ready_threaded(waiters, timeout=300):
def wait_for_ready_threaded(waiters, timeout=600):
threads = [
threading.Thread(target=waiter.wait_for_ready, daemon=True, args=(timeout,))
for waiter in waiters
Expand Down Expand Up @@ -558,7 +558,7 @@ def _all_resources_ready(namespace, timeout):
return wait_for_ready_threaded(waiters, timeout)


def wait_for_all_resources(namespace, timeout=300):
def wait_for_all_resources(namespace, timeout=600):
# wrap the other wait_fors in 1 wait_for so overall timeout is honored
# wait_for returns a tuple of the return code and the time taken
wait_for(
Expand All @@ -569,7 +569,7 @@ def wait_for_all_resources(namespace, timeout=300):
)


def wait_for_db_resources(namespace, timeout=300):
def wait_for_db_resources(namespace, timeout=600):
clowdapps = get_json("clowdapp", namespace=namespace).get("items", [])
if len(clowdapps) == 0:
raise ValueError(f"no clowdapps found in ns '{namespace}', no DB's to wait for")
Expand Down

0 comments on commit e4da6df

Please sign in to comment.