-
Notifications
You must be signed in to change notification settings - Fork 94
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
runahead limit not being cleared #4817
Comments
Investigating... |
I've failed to reproduce this - do you have the workflow you saw it in?
Pretty sure that's not supposed to happen. Tasks should get queued only on release from runahead. |
Yes, but I don't think it will help much (it's a beast). Here's a GUI screenshot showing a running task with a running job that is in the runahead state: If I see this in the wild I will try to grab some graph GraphQL output and test in Tui to check whether the issue is definitely data store side.
This was more of a "steps to reproduce" than a description of what happened at the implementation level. I meant that the task could be considered to be held back by both runahead limit and (likely multiple) queues, perhaps the interaction between the two could be the root cause. |
(might be fixed by recent changes: #4906 ?) |
This hasn't been seen in the wild for a while so closing this issue. Please re-open if this is spotted again. |
(see also #4037)
I have seen running ("naturally triggered") tasks appear in the GUI as runahead limited. We are probably missing a data store update somewhere causing the runahead status to get out of date. These tasks were probably both internally queued and runahead limited at the same time which might possibly be the cause of the issue?
Pull requests welcome!
This is an Open Source project - please consider contributing a bug fix
yourself (please read
CONTRIBUTING.md
before starting any work though).The text was updated successfully, but these errors were encountered: