Skip to content

Amended polling system to avoid polling succeeded, failed, or expired tasks - #7409

Draft
Scott-Owen-James wants to merge 5 commits into
cylc:masterfrom
Scott-Owen-James:7123
Draft

Amended polling system to avoid polling succeeded, failed, or expired tasks#7409
Scott-Owen-James wants to merge 5 commits into
cylc:masterfrom
Scott-Owen-James:7123

Conversation

@Scott-Owen-James

Copy link
Copy Markdown
Contributor

Amended polling system to avoid polling succeeded, failed, or expired tasks, closes #7123

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.

@oliver-sanders

Copy link
Copy Markdown
Member

I'm not sure if this change addresses the issue of polling "preparing" tasks?

@Scott-Owen-James

Scott-Owen-James commented Aug 3, 2026

Copy link
Copy Markdown
Contributor Author

I'm not sure if this change addresses the issue of polling "preparing" tasks?

This is embarrassing, somehow managed to miss it when I was adding other states to ignore, shouldn't be difficult to rectify!

Comment thread tests/integration/test_scheduler.py Outdated
@Scott-Owen-James
Scott-Owen-James marked this pull request as draft August 12, 2026 12:34
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.

poll: should not be able to poll preparing jobs

2 participants