-
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
graph parser: expire-all ? #5354
Comments
oliver-sanders
added
bug?
Not sure if this is a bug or not
question
Flag this as a question for the next Cylc project meeting.
labels
Feb 8, 2023
Would be helpful in my use case, so thanks for raising this issue @oliver-sanders. |
Even worse: #5364 |
Bumping back until #5364 is sorted. |
hjoliver
added
bug
Something is wrong :(
and removed
question
Flag this as a question for the next Cylc project meeting.
bug?
Not sure if this is a bug or not
labels
Mar 16, 2023
PR done, bumping the milestone forward again. |
This can now be attempted, however, won't necessarily work until #5364 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The
expire-all
trigger (valid at Cylc 7) appears to be missing from Cylc 8:cylc-flow/cylc/flow/graph_parser.py
Lines 105 to 116 in 76762bb
I'm not sure whether
expire-all
was "officially supported" or just a side-effect of the way the logic worked, however, it seems like a sensible trigger condition to me. I'm not sure whether its removal was intentional, nothing springs to mind.At Cylc 8 such a trigger would probably required a
?
modifier (as succeeded and expired are orthogonal), but should still work.Pull requests welcome!
The text was updated successfully, but these errors were encountered: