Skip to content

#6598 cat-log: new auto cat/tail mode#6913

Merged
wxtim merged 6 commits intocylc:masterfrom
ChrisPaulBennett:6598-cat-log-new-auto-cattail-mode
Oct 15, 2025
Merged

#6598 cat-log: new auto cat/tail mode#6913
wxtim merged 6 commits intocylc:masterfrom
ChrisPaulBennett:6598-cat-log-new-auto-cattail-mode

Conversation

@ChrisPaulBennett
Copy link
Copy Markdown
Contributor

@ChrisPaulBennett ChrisPaulBennett commented Aug 11, 2025

This PR closes #6598.
This adds a new 'auto' mode to the cylc cat-log command. When asking for job logs (Not workflow logs), the logic will determine if the workflow is live, tail will be used otherwise cat will be used.

Check List

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.
  • Contains logically grouped changes (else tidy your branch by rebase).
  • Does not contain off-topic changes (use other PRs for other changes).
  • Applied any dependency changes to both setup.cfg (and conda-environment.yml if present).
  • Tests are included (or explain why tests are not needed).
  • Changelog entry included if this is a change that can affect users
  • Cylc-Doc pull request opened if required at cylc/cylc-doc/pull/XXXX.
  • If this is a bug fix, PR should be raised against the relevant ?.?.x branch.

Copy link
Copy Markdown
Member

@oliver-sanders oliver-sanders left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@oliver-sanders oliver-sanders modified the milestones: 8.6.0, 8.7.0 Sep 1, 2025
@oliver-sanders
Copy link
Copy Markdown
Member

Poke @wxtim

@wxtim wxtim merged commit 79d50ad into cylc:master Oct 15, 2025
27 of 28 checks passed
hjoliver pushed a commit to hjoliver/cylc-flow that referenced this pull request Oct 21, 2025
wxtim pushed a commit to wxtim/cylc that referenced this pull request Nov 13, 2025
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 this pull request may close these issues.

cat-log: new auto cat/tail mode

3 participants