Tim Pillinger

So I think that in the end my decision (with Oliver's approval) was as follows: - Master after https://github.com/cylc/cylc-flow/pull/5721 and 8.2.x have now diverged significantly. - Almost no-one is using...

> ``` > tests/integration/tui/test_updater.py::test_filters - TypeError: 'NoneType' object is not subscriptable > ``` > > Looks genuine? Yes, it was both genuine and potentially nasty. Hopefully ok now, although it...

@oliver-sanders

> However `foo` fails again `fail try 1 only` != `fail submit 1 only`: It only plays with `execution retry delays`, not manual resubmits. I'm not sure if that's OK,...

> That's OK, but the docs could do with that detail > >  Agreed - see the commit immediately above this comment. 😄 (https://github.com/cylc/cylc-flow/pull/5721/commits/e9a087e77eca37b46573cbb6d4cdf7f25fd8ced0)

> Are the mode_settings are being created when the task is created (which would be immediately after 1/a submits in this case) rather than when the task is submitted (which...

> ``` > > ``` > > > > > > I tried setting `fail try 1 only=True` after `foo` had already retried a few times and got this Think...

> Broadcasting `[simulation]default run length=PT1S` causes the task to succeed 😕 Got it - side effect of earlier change.

> * Broadcasting `execution retry delays` does not cause a change when running in simulation mode I cannot replicate this on the command line. Given the context of your comment...

@MetRonnie - This is a good spot. I think that the broadcasts probably need to be applied in the `TaskJobManager._simulation_submit_task_jobs` method to ensure that we are broadcasting all task settings,...