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

cli: infer run number where possible #3895

Closed
oliver-sanders opened this issue Oct 27, 2020 · 3 comments · Fixed by #4285
Closed

cli: infer run number where possible #3895

oliver-sanders opened this issue Oct 27, 2020 · 3 comments · Fixed by #4285
Assignees
Milestone

Comments

@oliver-sanders
Copy link
Member

oliver-sanders commented Oct 27, 2020

Attempt after #3889, #4230

The default behaviour of cylc install will be to install workflows into an automatically named run (i.e. run<N>).

This means that the CLI will become a little cumbersome e.g.

# before
$ cylc stop my-flow
# after
$ cylc stop my-flow/run1

To combat this we should make the relevant Cylc subcommands (those with a REG argument) run-dir aware. That is to say that if the REG provided is not a workflow (denoted by the presence of a .service directory) but does contain automatically named runs (denoted by the presence of a runN symlink), then default to whatever run runN is pointing at.

This change returns the CLI REG argument to the way it was before for most cases whilst providing the ability to control named runs individually where required.

@MetRonnie
Copy link
Member

Just to emphasise the last point (more of a note to self really): named runs (with cylc install foo --run-name=bar) should not be inferred (i.e. cylc play foo should not do anything, you would have to explicitly do cylc play foo/bar)

@MetRonnie MetRonnie mentioned this issue Apr 9, 2021
7 tasks
@hjoliver hjoliver modified the milestones: cylc-8.0b1, cylc-8.0b2 Apr 15, 2021
@MetRonnie
Copy link
Member

Will attempt after #4180

@oliver-sanders
Copy link
Member Author

Bumped the centralisation of workflow argument parsing into a separate issue to make this one a little clearer - #4230

@MetRonnie MetRonnie changed the title cli: infer run name where possible cli: infer run number where possible Jun 30, 2021
@oliver-sanders oliver-sanders modified the milestones: cylc-8.0b2, cylc-8.0b3 Jul 28, 2021
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.

3 participants