Skip to content

Increase timeouts in test_asyncprocess::test_simple#6992

Closed
fjetter wants to merge 1 commit into
dask:mainfrom
fjetter:increase_timeouts_asyncproc_test_simple
Closed

Increase timeouts in test_asyncprocess::test_simple#6992
fjetter wants to merge 1 commit into
dask:mainfrom
fjetter:increase_timeouts_asyncproc_test_simple

Conversation

@fjetter

@fjetter fjetter commented Sep 2, 2022

Copy link
Copy Markdown
Member

This test flaked over in #6991

The functionality appeared to be OK but the process join took ~6 seconds which tripped the assert dt <= 1.0. I simply increased all timeouts a bit

@fjetter

fjetter commented Sep 2, 2022

Copy link
Copy Markdown
Member Author

ubu 310 failure due to a conda network problem

@fjetter

fjetter commented Sep 2, 2022

Copy link
Copy Markdown
Member Author

cc @dask/gpu gpuCI is failing for some reason. I doubt this is connected to this PR. Details don't show any info so maybe this can be ignored

@github-actions

github-actions Bot commented Sep 2, 2022

Copy link
Copy Markdown
Contributor

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 32m 33s ⏱️ + 13m 33s
  3 082 tests ±    0    2 998 ✔️ +    2       84 💤  -   1  0  - 1 
23 423 runs  +621  22 356 ✔️ +537  1 067 💤 +85  0  - 1 

Results for commit 891556a. ± Comparison against base commit acf6078.

@pentschev

Copy link
Copy Markdown
Member

Looks like a hiccup, it seems it didn't even start.

@pentschev

Copy link
Copy Markdown
Member

rerun tests

@pentschev

Copy link
Copy Markdown
Member

gpuCI is green now.

@fjetter if this happens again in the future, I think you also have permission to do rerun gpuCI by commenting the same as I did in #6992 (comment) (purposely avoiding to say that sentence here again to avoid it running once more).

@fjetter

fjetter commented Sep 2, 2022

Copy link
Copy Markdown
Member Author

I think you also have permission to do rerun gpuCI by commenting the same as I did

sure, will do

@fjetter fjetter closed this Jul 10, 2025
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.

2 participants