-
-
Notifications
You must be signed in to change notification settings - Fork 715
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
Set lower tick frequency in tests #5977
Conversation
@graingert it would be good to know if this alleviates the pain that you were seeing in #5908 if you have the time tomorrow |
distributed/utils_test.py
Outdated
dask.config.set( | ||
{ | ||
"distributed.comm.timeouts.connect": "5s", | ||
"distributed.admin.tick.interval": "500 ms", | ||
} | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This config setting is not side effect free, i.e. it doesn't reset itself properly afterwards. That causes tests to run with different configurations depending on the order of test execution.
I tried to fix it earlier but that causes many other failures. I'd prefer not adding more config params to this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was curious why we weren't using the context manager here. I'd be tempted to use it and see what happens. I'm going to push up a commit now to see what breaks. Probably I'm walking down a bad path, but we'll see what happens :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests seem happy here. Do you still have concerns? I wouldn't be surprised to learn that there was untested conflicts somewhere, but I'm hoping not :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if the tests are happy, I'm happy. this is how it is supposed to be after all
Possible alleviation of the pain mentioned in #5908