diff options
author | Bob Fournier <bfournie@redhat.com> | 2020-05-29 08:33:22 -0400 |
---|---|---|
committer | Bob Fournier <bfournie@redhat.com> | 2020-05-29 08:54:20 -0400 |
commit | 3ae7e6880da98318b26779828ff111d9cc893f65 (patch) | |
tree | e89dea65d456ecf833725b6b08ec6a18733f96cd /ironic/conf | |
parent | dac810aaf41612d0199cd87f900f5c365bc06b4a (diff) | |
download | ironic-3ae7e6880da98318b26779828ff111d9cc893f65.tar.gz |
Change default to ``False`` for ``use_ipmitool_retries``
A new configuration parameter was introduced in
https://review.opendev.org/#/c/731676/ to work around an
issue with some BMCs that don't support the Cipher
Suites command. This changes the default value to
``False`` so that Ironic will do the retries by default.
Note that this should not be backported to avoid changing
the behaviour in stable branches.
Change-Id: I34bf7e5d79defc23161213aa8942edace4b87b78
Story: 2007632
Task: 39676
Diffstat (limited to 'ironic/conf')
-rw-r--r-- | ironic/conf/ipmi.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ironic/conf/ipmi.py b/ironic/conf/ipmi.py index 1ac007c61..0ef39b7b2 100644 --- a/ironic/conf/ipmi.py +++ b/ironic/conf/ipmi.py @@ -38,13 +38,13 @@ opts = [ 'that setting this too low may cause the BMC to crash. ' 'Recommended setting is 5 seconds.')), cfg.BoolOpt('use_ipmitool_retries', - default=True, + default=False, help=_('When set to True and the parameters are supported by ' 'ipmitool, the number of retries and the retry ' 'interval are passed to ipmitool as parameters, and ' 'ipmitool will do the retries. When set to False, ' 'ironic will retry the ipmitool commands. ' - 'Recommended setting is True')), + 'Recommended setting is False')), cfg.BoolOpt('kill_on_timeout', default=True, mutable=True, |