Skip to content

Pre batch processing task state (Option 2)#6859

Closed
dwsutherland wants to merge 1 commit intocylc:masterfrom
dwsutherland:ds-pre-batch-proc-option2
Closed

Pre batch processing task state (Option 2)#6859
dwsutherland wants to merge 1 commit intocylc:masterfrom
dwsutherland:ds-pre-batch-proc-option2

Conversation

@dwsutherland
Copy link
Copy Markdown
Member

@dwsutherland dwsutherland commented Jul 18, 2025

closes #6567, closes #6860, closes #6861

Note: The following view, will almost never been seen (otherwise TUI would've be crashing all the time previously), and has been generated by commenting out this batch block:
image

With that said, momentarily the pre batch processed DB would show:
image

(The WUI should change the ICON to (?) or something appropriate)

description

    TASK_STATUS_UNKNOWN:
        'Job status yet to be determined.'

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.

A stateless task is a data store bug.

It's possible to smooth it over in the UIs, but we should really be fixing the problem at the source.

@oliver-sanders
Copy link
Copy Markdown
Member

Really don't want to go down the route of introducing an "unknown" task state to cover up a bug.

But strangely, I've managed to reproduce the bug on this branch too. Something stranger is at work...

I'll close this to help tidy up open PRs.

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.

data store: blank task state

2 participants