Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

8.0.x: bugfixes #5064

Merged
merged 4 commits into from
Aug 12, 2022
Merged

8.0.x: bugfixes #5064

merged 4 commits into from
Aug 12, 2022

Conversation

oliver-sanders
Copy link
Member

Latest batch of bugfixes.

oliver-sanders and others added 4 commits August 8, 2022 15:41
* 8.0.x version of #5040
* Closes #4866
* Reload causes new instances of each task to be created.
* The attributes of the old instances are copied onto the new ones.
* Xtriggers were not copied so were lost.
* This means that all xtriggers were effectively deleted on reload.
* Fix lack of abort for main loop auto restart in no-detach mode

* Tidy & add type annotations

* Fix flaky spurious log rollover on auto restart

* Remember verbosity on auto restart

* Prevent exceptions during auto restart from being swallowed

* Update changelog

* De-flake test
* The workflow_params function was using the private DB to list workflow
  params.
* Really it should be using the public DB as only the scheduler
  process/thread should be accessing the private DB to avoid the
  potential for DB locking.
@oliver-sanders oliver-sanders added this to the cylc-8.1.0 milestone Aug 12, 2022
@oliver-sanders oliver-sanders self-assigned this Aug 12, 2022
@oliver-sanders oliver-sanders changed the title 8.0.x 8.0.x: bugfixes Aug 12, 2022
@MetRonnie MetRonnie merged commit 138ec26 into master Aug 12, 2022
@MetRonnie MetRonnie added the sync PR for synchronising changes between 8.*.x and master label Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sync PR for synchronising changes between 8.*.x and master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants