From 323344e07cb5cdcbb99a661a40f5b3951a8839d1 Mon Sep 17 00:00:00 2001 From: Arne Wiebalck Date: Thu, 4 Nov 2021 17:57:59 +0100 Subject: [PATCH] [Trivial] Clarify conditions under which power recovery is attempted Be more precise when describing the conditions for automatic recovery from power failures ('maintenance type' is a term we use nowhere else). Change-Id: Iaf14c0fc73f8c97b9d8669485011966a650c21a8 --- ironic/conductor/manager.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ironic/conductor/manager.py b/ironic/conductor/manager.py index 4c49bc7893..7b91996171 100644 --- a/ironic/conductor/manager.py +++ b/ironic/conductor/manager.py @@ -1512,9 +1512,10 @@ class ConductorManager(base_manager.BaseConductorManager): conditions are met: 1) Node is mapped to this conductor. - 2) Node is in maintenance with maintenance type of power failure. - 3) Node is not reserved. - 4) Node is not in the ENROLL state. + 2) Node is in maintenance. + 3) Node's fault field is set to 'power failure'. + 4) Node is not reserved. + 5) Node is not in the ENROLL state. """ def handle_recovery(task, actual_power_state): """Handle recovery when power sync is succeeded."""