openstack/ironic-inspector
Code Issues Proposed changes

PXE filter options have no effect yet

Browse Source 
Updating the CONTRIBUTING.rst, config.py and example.conf to inform the
Operator (and Developer) that the PXE filter options have no effect yet.

Change-Id: I032114934e300af1e1908369552f1b06c939002f
This commit is contained in:
dparalen 2017-08-15 15:12:32 +02:00
parent 519112c737
commit 552b3ef946
3 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
CONTRIBUTING.rst
View File

@ -320,6 +320,8 @@ the database::
Implementing PXE Filter Drivers Implementing PXE Filter Drivers
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
.. note:: It is not possible yet to use PXE filter drivers.
Background Background
---------- ----------

5
example.conf
View File

@ -789,11 +789,12 @@
# From ironic_inspector # From ironic_inspector
# #
# PXE boot filter driver to use, such as iptables (string value) # PXE boot filter driver to use, such as iptables. This option has no
# effect yet. (string value)
#driver = noop #driver = noop
# Amount of time in seconds, after which repeat periodic update of the # Amount of time in seconds, after which repeat periodic update of the
# filter. (integer value) # filter. This option has no effect yet. (integer value)
# Minimum value: 0 # Minimum value: 0
#sync_period = 15 #sync_period = 15

5
ironic_inspector/conf.py
View File

@ -190,10 +190,11 @@ SERVICE_OPTS = [
PXE_FILTER_OPTS = [ PXE_FILTER_OPTS = [
cfg.StrOpt('driver', default='noop', cfg.StrOpt('driver', default='noop',
help=_('PXE boot filter driver to use, such as iptables')), help=_('PXE boot filter driver to use, such as iptables. '
'This option has no effect yet.')),
cfg.IntOpt('sync_period', default=15, min=0, cfg.IntOpt('sync_period', default=15, min=0,
help=_('Amount of time in seconds, after which repeat periodic ' help=_('Amount of time in seconds, after which repeat periodic '
'update of the filter.')), 'update of the filter. This option has no effect yet.')),
] ]
cfg.CONF.register_opts(SERVICE_OPTS) cfg.CONF.register_opts(SERVICE_OPTS)