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

Test CI #8943

Closed
wants to merge 2 commits into from
Closed

Test CI #8943

wants to merge 2 commits into from

Conversation

jrbourbeau
Copy link
Member

I'm seeing some weird failures elsewhere and am wanting to see if they're on main too

@jrbourbeau jrbourbeau added the skip-caching Apply to PRs to disable CI environment caching label Nov 19, 2024
Copy link
Contributor

Unit Test Results

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

    25 files  ±0      25 suites  ±0   10h 10m 18s ⏱️ - 31m 23s
 4 130 tests ±0   3 975 ✅ +118    110 💤 ±0   45 ❌  - 117 
47 692 runs   - 5  45 131 ✅ +803  2 121 💤 ±0  440 ❌  - 803 

For more details on these failures, see this check.

Results for commit bb25546. ± Comparison against base commit 750cb91.

@jrbourbeau
Copy link
Member Author

CI is pretty broken here. @fjetter @phofl any thoughts on what might have happened?

I'm seeing lots of

RuntimeError: P2P d9513d99181485ac4237b30c4b28d743 failed during transfer phase

failures. Maybe the recursion stuff from yesterday #8920 ?

@jrbourbeau
Copy link
Member Author

@phofl mentioned offline that dask/dask#11543 and #8944 will fix those failures

@fjetter
Copy link
Member

fjetter commented Nov 20, 2024

There is generally no need to open a PR like this to test CI. This is likely not the most reliable way anyhow. I recommend looking at https://dask.github.io/distributed/test_short_report.html

@fjetter fjetter closed this Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-caching Apply to PRs to disable CI environment caching
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants