diff options
| author | Devananda van der Veen <devananda.vdv@gmail.com> | 2014-05-30 11:59:54 -0700 |
|---|---|---|
| committer | Devananda van der Veen <devananda.vdv@gmail.com> | 2014-06-13 11:01:29 -0700 |
| commit | 12ef2bc621f6c713524d54a157bc3fe216b04977 (patch) | |
| tree | 0468fe70d4647b68bb66b8a7c6b6ab6a48b0d7ab /etc | |
| parent | 8b61c44fe2698d0ef2787d5e411b6332dcb6aae7 (diff) | |
| download | ironic-12ef2bc621f6c713524d54a157bc3fe216b04977.tar.gz | |
Let ipmitool natively retry commands
Instead of calling ipmitool multiple times on failure via
utils.execute(*args, attempts=3)
allow ipmitool to use its own native retry behavior with -N.. -R..
if those options are supported by the installed version of ipmitool.
This will fall back to a single run of ipmitool on older versions,
which should be fine -- it defaults to retry several times anyway.
This patch adds a configurable min time between retries, which is used,
in conjunction with the ipmi retry time, to determine these option's
values. It will be further leveraged in a subsequent patch as well.
It also adds a note in the deployer docs about known issues with
the openipmi project.
Change-Id: I7a4ff941144a03bd441459561efb68760391da1a
Partial-bug: #1320513
Diffstat (limited to 'etc')
| -rw-r--r-- | etc/ironic/ironic.conf.sample | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/etc/ironic/ironic.conf.sample b/etc/ironic/ironic.conf.sample index dd8396fec..7affe8c75 100644 --- a/etc/ironic/ironic.conf.sample +++ b/etc/ironic/ironic.conf.sample @@ -706,6 +706,12 @@ # value) #retry_timeout=60 +# Minimum time, in seconds, between IPMI operations sent to a +# server. There is a risk with some hardware that setting this +# too low may cause the BMC to crash. Recommended setting is 5 +# seconds. (integer value) +#min_command_interval=5 + [keystone_authtoken] |