Skip to content

expiry: clarify late expiry events - #7505

Open
oliver-sanders wants to merge 1 commit into
cylc:masterfrom
oliver-sanders:doc-699
Open

oliver-sanders wants to merge 1 commit into
cylc:masterfrom
oliver-sanders:doc-699

Conversation

@oliver-sanders

Copy link
Copy Markdown
Member

Goes with cylc/cylc-doc#974

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.

* Addresses cylc/cylc-doc#974
* Add some text to the examples section explaining when expiry happens.
@oliver-sanders oliver-sanders added this to the 8.7.x milestone Oct 2, 2026
@oliver-sanders oliver-sanders self-assigned this Oct 2, 2026
@oliver-sanders oliver-sanders added small doc Documentation labels Oct 2, 2026
Comment on lines +91 to +94

If you need something in your workflow to happen at a configured time, use
:term:`clock triggers <clock trigger>` rather than triggering off of the
``:expired`` output.

@hjoliver hjoliver Oct 5, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
If you need something in your workflow to happen at a configured time, use
:term:`clock triggers <clock trigger>` rather than triggering off of the
``:expired`` output.

I'd suggest not putting this here, unless you have evidence that users try to use expiry for this purpose? Triggering off of an expiry event seems an odd way to attempt anything other than respond to the expiry itself (i.e. "this task expired, it's not going to run, so do this instead").

Besides, clock triggers actually have a similar "lateness" problem to expiry - they aren't checked until the dependent task enters the active window, so clock triggers make things happen after, not at, a configured time.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Documentation small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants