openstack
/
ironic
Code Issues Proposed changes

Do not disable inspector periodic tasks if [inspector]enabled is False 

Our help strings claim that [inspector]enabled only affects classic drivers.
However, it also disables the inspector periodic tasks, if not set to True.

Change-Id: I2e3deaffa31259793f738dc77af49d5cb57af697
Story: #2002730
Task: #22579
(cherry picked from commit 2e568bdd56)
This commit is contained in:
Dmitry Tantsur 2018-06-27 16:02:23 +02:00
parent ea24d189a0
commit b3652391f3
3 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ironic/drivers/modules/inspector.py
View File

@ -130,8 +130,7 @@ class Inspector(base.InspectInterface):
eventlet.spawn_n(_start_inspection, task.node.uuid, task.context)
return states.INSPECTING
@periodics.periodic(spacing=CONF.inspector.status_check_period,
enabled=CONF.inspector.enabled)
@periodics.periodic(spacing=CONF.inspector.status_check_period)
def _periodic_check_result(self, manager, context):
"""Periodic task checking results of inspection."""
filters = {'provision_state': states.INSPECTING}

3
ironic/drivers/modules/oneview/inspect.py
View File

@ -63,8 +63,7 @@ class OneViewInspect(inspector.Inspector):
)
return super(OneViewInspect, self).inspect_hardware(task)
@periodics.periodic(spacing=CONF.inspector.status_check_period,
enabled=CONF.inspector.enabled)
@periodics.periodic(spacing=CONF.inspector.status_check_period)
def _periodic_check_result(self, manager, context):
filters = {'provision_state': states.INSPECTING}
node_iter = manager.iter_nodes(filters=filters)

8
releasenotes/notes/inspector-periodics-34449c9d77830b3c.yaml Normal file
View File

@ -0,0 +1,8 @@
---
fixes:
- |
The periodic tasks for the ``inspector`` inspect interface are no longer
disabled if the ``[inspector]enabled`` option is not set to ``True``.
The help string of this option claims that it does not apply to hardware
types. In any case, the periodic tasks are only run if any enabled
classic driver or hardware interface requires them.