Ronnie Dutta
Ronnie Dutta
(Figured out that the tests were passing locally for me because the `comm` command can't tell the difference between `✓` and `⨯` when `LANG=en_US.UTF-8` for some bizarre reason)
> Sounds like it would make sense to set `LANG=C` for the purpose of this comparison? #6183
N.B. will conflict with https://github.com/cylc/cylc-flow/pull/6640
@wxtim Can you please explain the bug in the PR description
> I suspect it's an issue with the test? Maybe the setting of the time zone isn't quite right? The timezone change in the test looks right to me. The...
Do we want a changelog entry for this one?
According to the warnings: > The event loop scope for asynchronous fixtures will default to the fixture caching scope. Future versions of pytest-asyncio will default the loop scope for asynchronous...
Ack, just noticed this regression: Cylc 8.2.7: ``` $ cylc workflow-state Murphid -t foo -p 2001 -S catacomb --max-polls 1 InputError: invalid status 'catacomb' ``` Cylc 8.3.1 and this branch:...
Would it be worth having a deprecation warning until 8.4 and then making it an InputError (either for the CLI or both CLI and xtrigger)? Although I don't think it...
This would appear to be caused by a task existing in the task pool but not the data store: https://github.com/cylc/cylc-flow/blob/4db73a2061f93a4b72dafbf85efd38cce8f570e2/cylc/flow/data_store_mgr.py#L1801-L1809 With `self.store_node_fetcher(tokens)` returning `_, False`. This shouldn't be possible, maybe...