From 31c7469a91c58235b9200ce417ace57447bf23a7 Mon Sep 17 00:00:00 2001 From: Julia Kreger Date: Wed, 11 Mar 2020 17:19:49 -0700 Subject: [PATCH] Extend power sync timeout for Ericsson SDI Ericsson SDI uses cached power state data in what is returned to the API user, and typically this takes longer than 30 seconds to provide an updated value to the API user. As such, if we do not extend the timeout value, such users of Ironic's Redfish API interface will have deployments fail as the power state timeout will be encountered. This change landed in ussuri, and is proposed to stable/train due to TripleO CI failures. Related-Bug: #1947403 Change-Id: I0aa5131504b60b13d43c73c9a3be1f50f7855cbc (cherry picked from commit 4383303dfbfe15f3dfbd8be8d4cc6d70aa703019) --- ironic/conf/conductor.py | 2 +- ...or-power-sync-timeout-extension-fa5e7b5fdd679d84.yaml | 9 +++++++++ 2 files changed, 10 insertions(+), 1 deletion(-) create mode 100644 releasenotes/notes/conductor-power-sync-timeout-extension-fa5e7b5fdd679d84.yaml diff --git a/ironic/conf/conductor.py b/ironic/conf/conductor.py index 3ed0fc4f45..8b37b54679 100644 --- a/ironic/conf/conductor.py +++ b/ironic/conf/conductor.py @@ -209,7 +209,7 @@ opts = [ help=_('Timeout (in seconds) of soft reboot and soft power ' 'off operation. This value always has to be positive.')), cfg.IntOpt('power_state_change_timeout', - min=2, default=30, + min=2, default=60, help=_('Number of seconds to wait for power operations to ' 'complete, i.e., so that a baremetal node is in the ' 'desired power state. If timed out, the power operation ' diff --git a/releasenotes/notes/conductor-power-sync-timeout-extension-fa5e7b5fdd679d84.yaml b/releasenotes/notes/conductor-power-sync-timeout-extension-fa5e7b5fdd679d84.yaml new file mode 100644 index 0000000000..7696bfb602 --- /dev/null +++ b/releasenotes/notes/conductor-power-sync-timeout-extension-fa5e7b5fdd679d84.yaml @@ -0,0 +1,9 @@ +--- +other: + - | + The ``[conductor]power_state_change_timeout`` default value has been + extended to ``60`` seconds from ``30`` seconds. This is due to some + API interfaces with Redfish, may cache the power state and thus may + take longer than thirty seconds to update after a change has been + requested. Please see `here `_ + for more information.