Skip to content

Commit 96b9429

Browse files
Apply suggestions from code review
Co-authored-by: Hilary James Oliver <hilary.j.oliver@gmail.com>
1 parent c5d56f8 commit 96b9429

File tree

2 files changed

+9
-10
lines changed

2 files changed

+9
-10
lines changed

changes.d/6623.fix.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
auto restart: The "force condemn" option (that tells workflows running on a
1+
Auto restart: The "force condemn" option (that tells workflows running on a
22
server to shutdown as opposed to migrate) hasn't worked with the host-selection
33
mechanism since Cylc 8.0.0. This has now been fixed and the "force condemn"
44
option has been restored in the documentation.

cylc/flow/cfgspec/globalcfg.py

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -828,23 +828,22 @@ def default_for(
828828
Conf('condemned', VDR.V_ABSOLUTE_HOST_LIST, desc=f'''
829829
List run hosts that workflows should *not* run on.
830830
831-
Any hosts listed here will be subtracted from the
832-
`available <global.cylc[scheduler][run hosts]>`
833-
hosts:
831+
These will be subtracted from the
832+
`available <global.cylc[scheduler][run hosts]>`:
834833
835834
* Workflows will not start on condemned hosts.
836835
* Workflows that are running on condemned hosts will attempt
837-
to migrate to an uncondemned host (providing the
836+
to migrate to an available host (providing the
838837
`auto restart
839838
<global.cylc[scheduler][main loop][auto restart]>`
840839
plugin is enabled).
841840
842841
This feature can be used to drain a host for patching, or
843-
remove a host that is surplus to requirement.
842+
remove a host that is surplus to requirements.
844843
845-
Hostnames listed here may be followed by a ``!`` character.
846-
This activates "force mode", workflows running on a
847-
force-condenmed host will shutdown rather than attempting to
844+
If a hostname listed here is followed by a ``!`` character
845+
("force mode"), workflows running on it
846+
will shutdown rather than attempting to
848847
migrate (providing the
849848
`auto restart
850849
<global.cylc[scheduler][main loop][auto restart]>` plugin
@@ -871,7 +870,7 @@ def default_for(
871870
872871
.. versionchanged:: 8.4.2
873872
874-
The "force-condemn" option caused issues at workflow
873+
The force-condemn ("!") option caused issues at workflow
875874
startup for Cylc versions between 8.0.0 and 8.4.1
876875
inclusive.
877876

0 commit comments

Comments
 (0)