diff options
author | Dmitry Tantsur <dtantsur@protonmail.com> | 2021-01-08 14:48:54 +0100 |
---|---|---|
committer | Dmitry Tantsur <dtantsur@protonmail.com> | 2021-01-08 14:57:07 +0100 |
commit | ad696c9bac379a690b1f674de4983bf1a06a243e (patch) | |
tree | 7d887fbe2167da80d0cffd56535c9f56c125638d /doc | |
parent | d5f184ea16fc2355ff9d940465fcd3de9c8be30c (diff) | |
download | ironic-ad696c9bac379a690b1f674de4983bf1a06a243e.tar.gz |
Do not enter maintenance if cleaning fails before running the 1st step
We use maintenance mode to signal that hardware needs additional
intervention, because of potential damage or stuck long-running
processes. This is not the case for PXE booting or invalid requested
manual clean steps, so don't set maintenance if no clean step is
running when the failure occurs.
Change-Id: I8a7ce072359660fc6640e5f20ec2d3c452033557
Diffstat (limited to 'doc')
-rw-r--r-- | doc/source/admin/cleaning.rst | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/doc/source/admin/cleaning.rst b/doc/source/admin/cleaning.rst index 8338a8b17..9cfd628c7 100644 --- a/doc/source/admin/cleaning.rst +++ b/doc/source/admin/cleaning.rst @@ -314,9 +314,15 @@ cleaning. Troubleshooting =============== -If cleaning fails on a node, the node will be put into ``clean failed`` state -and placed in maintenance mode, to prevent ironic from taking actions on the -node. +If cleaning fails on a node, the node will be put into ``clean failed`` state. +If the failure happens while running a clean step, the node is also placed in +maintenance mode to prevent ironic from taking actions on the node. The +operator should validate that no permanent damage has been done to the +node and no processes are still running on it before removing the maintenance +mode. + +.. note:: Older versions of ironic may put the node to maintenance even when + no clean step has been running. Nodes in ``clean failed`` will not be powered off, as the node might be in a state such that powering it off could damage the node or remove useful |