-
-
Notifications
You must be signed in to change notification settings - Fork 328
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
Runtime: Always wake runner immediately once a new task is enqueued #375
Conversation
@joell Can you confirm that this fixes your issue? It won't repro for me anymore, at least.. @clux I know we don't really do LTS, but maybe it's worth backporting this to 0.45.1 or something, since it's pretty critical for controllers, and 0.45 will presumably be the final release to support Tokio 0.2 (which people might be stuck on for Legacy Reasons). |
@teozkr : Sure, if this fixes it I can cherry-pick this to a |
Planning on getting this in a minor today. |
Released in kube* |
@teozkr : i tried cherry picking this commit from the last 0.46.0 commit into a new branch, but it's already using tokio 1.0 features. The import reordering of tokio imports + rename of sleep -> delay_for was fine, but tests requires |
@clux The 0.2 equivalent should be |
|
Fixes #372