openstack
/
ironic-inspector
Code Issues Proposed changes

Add drac_address to ipmi_address_fields 

Since Ia23e8582a398dca9ca11762ee6fe1789fdba9777 ironic's DRAC drivers
deprecated 'drac_host' driver_info field in favor of 'drac_address'.

This patch adds 'drac_address' to default value for 'ipmi_address_fields'
config option so that inspector can support both nodes with old and new
driver_info field.

Change-Id: I90559d7a99462cd0de2a0c8ce047015955b8cfda
Related-Bug: #1644210
This commit is contained in:
Pavlo Shchelokovskyy 2016-12-15 16:34:31 +02:00
parent fe7b4e3267
commit 208088186b
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
example.conf
View File

@ -64,7 +64,7 @@
# Ironic driver_info fields that are equivalent to ipmi_address. (list
# value)
#ipmi_address_fields = ilo_address,drac_host,cimc_address
#ipmi_address_fields = ilo_address,drac_host,drac_address,cimc_address
# Path to the rootwrap configuration file to use for running commands
# as root (string value)

3
ironic_inspector/conf.py
View File

@ -208,7 +208,8 @@ SERVICE_OPTS = [
'expression will be affected by introspection_delay '
'setting.')),
cfg.ListOpt('ipmi_address_fields',
default=['ilo_address', 'drac_host', 'cimc_address'],
default=['ilo_address', 'drac_host', 'drac_address',
'cimc_address'],
help=_('Ironic driver_info fields that are equivalent '
'to ipmi_address.')),
cfg.StrOpt('rootwrap_config',