Skip to content
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

reset-to-waiting when prerequisite tasks have been removed #2654

Closed
hjoliver opened this issue May 3, 2018 · 1 comment · Fixed by #3515
Closed

reset-to-waiting when prerequisite tasks have been removed #2654

hjoliver opened this issue May 3, 2018 · 1 comment · Fixed by #3515
Assignees
Milestone

Comments

@hjoliver
Copy link
Member

hjoliver commented May 3, 2018

I want to reset the prerequisites of a whole sub-graph of tasks so that they all run again in the right order (as opposed to triggering all the tasks immediately). [Note I can't explicitly refer to a sub-graph yet - see #1314 - but I can achieve the same thing by resetting an appropriately defined family]. However, if I do this I might find that everything stalls because the prerequisites of the task(s) at the top of the sub-graph(s) have already been removed from the suite.

This may be confusing to users: the (removed) tasks did in fact already run, so why are the associated prerequisites not satisfied? To understand this you have to understand the life cycle of task proxies, but as per #2143 users should not need to know that.

How to do what the user wants:

If a group of one or more tasks gets "reset to waiting", analyse the prerequisites of each one. If a prerequisite is a member of the group, reset it to not-satisfied. If it is not a member of the group and the associated task proxy has been removed from the suite, interrogate the database to see if it already completed (and set the prerequisite satisfied if it did complete).

@hjoliver hjoliver added this to the some-day milestone May 3, 2018
@hjoliver hjoliver modified the milestones: some-day, cylc-8.0.0 Aug 4, 2021
@hjoliver hjoliver self-assigned this Aug 4, 2021
@hjoliver
Copy link
Member Author

hjoliver commented Aug 4, 2021

Closed by spawn-on-demand #3515

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants