Ronnie Dutta
Ronnie Dutta
From that PR: > I think we will want to add the "flow label" into the `tasks_to_hold` set in order to handle the situation where we have the same task...
Something's gone wrong git-wise
``` [scheduler] [[events]] abort on workflow timeout = True workflow timeout = PT1S abort handlers = echo "ALPHA" workflow timeout handlers = echo "TANGO" ``` The workflow timeout handler is...
I think he used the commit message as the PR title and it got truncated, continuing on the body. I updated the title to something a little more readable
Tests need updating
I can't reproduce with your original example ``` INFO - [1/foo preparing job:01 flows:1] => submitted INFO - [command] reload_workflow INFO - PAUSING the workflow now: Reloading workflow INFO -...
Everything now working as expected for me on #5658, no unexpected warnings
TUI integration test failures are unrelated to your changes - see #5849 However there are a few functional test failures here
> However there are a few functional test failures here Looks like it was the `ValueError` vs `KeyError` thing. Functional tests passing now 👍