Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix flaky test_gather_dep_one_worker_always_busy #6554

Merged
merged 2 commits into from
Jun 10, 2022

Conversation

crusaderky
Copy link
Collaborator

@crusaderky crusaderky commented Jun 9, 2022

Closes #6533
The test was failing because the CI host would occasionally not fit at least 3 runs of PeriodicCallback(150ms) + network comms in a 500ms window.
Now relaxed to at least 2 runs in a 800ms window.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 9, 2022

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

       15 files  ±0         15 suites  ±0   6h 22m 2s ⏱️ - 12m 30s
  2 856 tests ±0    2 774 ✔️ +1    81 💤 ±0  1 ±0 
21 158 runs   - 1  20 211 ✔️  - 5  946 💤 +5  1 ±0 

For more details on these failures, see this check.

Results for commit 07860b3. ± Comparison against base commit 43ca938.

@crusaderky crusaderky merged commit 6272e20 into dask:main Jun 10, 2022
@crusaderky crusaderky deleted the busy_gather_pc branch June 10, 2022 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

flaky test_gather_dep_one_worker_always_busy
1 participant