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

install: report if previous run is still running #5127

Closed
oliver-sanders opened this issue Sep 12, 2022 · 2 comments · Fixed by #5184
Closed

install: report if previous run is still running #5127

oliver-sanders opened this issue Sep 12, 2022 · 2 comments · Fixed by #5184
Assignees
@hjoliver
Labels
small
Milestone
cylc-8.1.0

Comments

@oliver-sanders
Copy link
Member

If a previous instance of a workflow is still running when a new one is installed it would be good for cylc install to detect this and inform the user.

E.G. something along the lines of:

INSTALLED ....

whatever/run1 is still running, to shut it down (killing active tasks)
$ cylc stop whatever/run1 --kill

I don't think this should block the new installation, just a quick message after it has completed.

Pull requests welcome!
@oliver-sanders oliver-sanders added this to the cylc-8.x milestone Sep 12, 2022
@oliver-sanders oliver-sanders changed the title install: report of previous run is still running install: report if previous run is still running Sep 12, 2022
@oliver-sanders
Copy link
Member Author

oliver-sanders commented Oct 3, 2022 •
edited
Loading

Have seen quite a few cases of this which raises the priority somewhat. An especially likely mistake for people migrating from rose suite-run. Can be considered a UX bug.

@oliver-sanders oliver-sanders modified the milestones: cylc-8.x, cylc-8.1.0 Oct 5, 2022
@oliver-sanders
Copy link
Member Author

Bumped up to 8.1.0 as this is looking increasingly important.

@hjoliver hjoliver self-assigned this Oct 10, 2022
@hjoliver hjoliver mentioned this issue Oct 10, 2022
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hjoliver hjoliver

Labels
small
Projects
None yet
Milestone
cylc-8.1.0
Development

Successfully merging a pull request may close this issue.

Scan workflow name during install. hjoliver/cylc-flow
2 participants
@hjoliver @oliver-sanders