Skip to content

Commit

Permalink
Fixup test_worker_doesnt_await_task_completion (#5979)
Browse files Browse the repository at this point in the history
This test was designed to verify that we don't wait for a 100s sleep
call to finish before terminating the workers.  However the time
provided, 5s, is about what it takes to restart anyway.  We now extend
this time to 20s, which is safely below 100s, and also safely above the
worker start time.

In the future we should figure out why it's taking us 5s to start up.
This is drifting upwards
  • Loading branch information
mrocklin authored Mar 23, 2022
1 parent 4cf1375 commit e8c3eac
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion distributed/tests/test_failed_workers.py
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,7 @@ def test_worker_doesnt_await_task_completion(loop):
start = time()
c.restart()
stop = time()
assert stop - start < 5
assert stop - start < 20


@pytest.mark.skipif(COMPILED, reason="Fails with cythonized scheduler")
Expand Down

0 comments on commit e8c3eac

Please sign in to comment.