-
-
Notifications
You must be signed in to change notification settings - Fork 717
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_ready_remove_worker
broken on Python 3.11
#8054
Comments
Ah you beat me two it. |
Timing and topic point to #7698. If a resolution is not trivial, maybe we should revert it for the time being? |
Regarding the P2P errors, I've described the problem over here: #8011 |
I don't want to revert #7698 just because of a couple of leaked connections. This PR has a significant real world impact and I'd rather temporarily disable the connection leak detection that reverting this PR |
Works for me, after an initial investigation the leak doesn't seem to be trivial to fix. |
It looks #7698 broke
test_ready_remove_worker
on Python 3.11:This test fails with
RuntimeError: ('Unclosed Comms', [<TCP (closed) ConnectionPool local=tcp://127.0.0.1:62390 remote=tcp://127.0.0.1:62381>, <TCP (closed) ConnectionPool local=tcp://127.0.0.1:62391 remote=tcp://127.0.0.1:62381>])
.(https://github.com/dask/distributed/actions/runs/5690294964/job/15423369445#step:19:1681)
cc @graingert
The text was updated successfully, but these errors were encountered: