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.

Sign up for GitHub

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

Jump to bottom

subtle-ish restart --hold bug #958

Closed
hjoliver opened this issue May 12, 2014 · 1 comment · Fixed by #1276
Closed

subtle-ish restart --hold bug #958

hjoliver opened this issue May 12, 2014 · 1 comment · Fixed by #1276
Assignees
@hjoliver
Labels
bug Something is wrong :(
Milestone
cylc-6.2.0

Comments

@hjoliver
Copy link
Member

Note that doing a restart with the --hold flag can now require manually triggering once the hold is released. This is because releasing the suite resets the 'held' tasks to 'waiting', and that causes its prerequisites to be reset back to unsatisfied. The problem is that (as of recently-ish) suites can shut down with tasks in the 'ready' state. These are held on restart if --hold is used, to stop them running immediately - but they should be set back to 'ready' not 'waiting' when the hold is released.
@hjoliver hjoliver added the bug label May 12, 2014
@matthewrmshin matthewrmshin added this to the soon milestone May 12, 2014
@matthewrmshin
Copy link
Contributor

Is this fixed by #1268 and/or #1271?

matthewrmshin added a commit to matthewrmshin/cylc-flow that referenced this issue Dec 30, 2014
@matthewrmshin
Test for cylc#958
a754bd3 
Stop suite --now while a task is in ready state. Restart with --hold and
then release. The ready task should continue instead of being set to
waiting.
@matthewrmshin matthewrmshin mentioned this issue Dec 30, 2014
Merged
@matthewrmshin matthewrmshin modified the milestones: next release, soon Dec 30, 2014
hjoliver added a commit that referenced this issue Jan 11, 2015
@hjoliver
Merge pull request #1276 from matthewrmshin/t-958
e5cb3e7 
Test for #958
@MetRonnie MetRonnie mentioned this issue Jan 28, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hjoliver hjoliver

Labels
bug Something is wrong :(
Projects
None yet
Milestone
cylc-6.2.0
Development

Successfully merging a pull request may close this issue.

Test for #958 matthewrmshin/cylc-flow
2 participants
@matthewrmshin @hjoliver