diff options
author | Julia Kreger <juliaashleykreger@gmail.com> | 2020-03-11 17:19:49 -0700 |
---|---|---|
committer | Steve Baker <sbaker@redhat.com> | 2021-10-27 13:01:51 +1300 |
commit | 31c7469a91c58235b9200ce417ace57447bf23a7 (patch) | |
tree | f8ca84e87feeb0edc4216516589f298d2b562149 | |
parent | bf156e679e88e23e7726d2a728e51c0c5d2f8bb5 (diff) | |
download | ironic-31c7469a91c58235b9200ce417ace57447bf23a7.tar.gz |
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)
-rw-r--r-- | ironic/conf/conductor.py | 2 | ||||
-rw-r--r-- | releasenotes/notes/conductor-power-sync-timeout-extension-fa5e7b5fdd679d84.yaml | 9 |
2 files changed, 10 insertions, 1 deletions
diff --git a/ironic/conf/conductor.py b/ironic/conf/conductor.py index 3ed0fc4f4..8b37b5467 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 000000000..7696bfb60 --- /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 <https://github.com/metal3-io/ironic-image/issues/143>`_ + for more information. |