Skip to content

Commit

Permalink
Merge pull request #1678 from globus/increase_test_timeout-sc-36175
Browse files Browse the repository at this point in the history
increase smoke_test timeout for intermittent failure
  • Loading branch information
LeiGlobus authored Oct 8, 2024
2 parents 0e2744d + c56c9d0 commit 7802643
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions smoke_tests/tests/test_running_functions.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,9 @@ def test_executor(compute_client, endpoint, tutorial_function_id):
num_tasks = 10
submit_count = 2 # we've had at least one bug that prevented executor re-use

# 30s sometimes fails, trying with 60 to see if that makes a difference
timeout_s = 60

# use client on newer versions and funcx_client on older
try:
gce = Executor(endpoint_id=endpoint, client=compute_client)
Expand All @@ -100,7 +103,7 @@ def test_executor(compute_client, endpoint, tutorial_function_id):
]

results = []
for f in concurrent.futures.as_completed(futures, timeout=30):
for f in concurrent.futures.as_completed(futures, timeout=timeout_s):
results.append(f.result())

assert (
Expand All @@ -114,7 +117,7 @@ def test_executor(compute_client, endpoint, tutorial_function_id):
assert len(futures) == submit_count * num_tasks

results = []
for f in concurrent.futures.as_completed(futures, timeout=30):
for f in concurrent.futures.as_completed(futures, timeout=timeout_s):
results.append(f.result())
assert all(
"Hello World!" == item for item in results
Expand Down

0 comments on commit 7802643

Please sign in to comment.