openstack/ironic
Code Issues Proposed changes

Clean up driver_factory.enabled_supported_interfaces

Browse Source 
This had an extra variable (and related work) that was unnecessary,
clean it up.

Change-Id: I245946a36e891d305dd06dfd3944a6e623d731a4
Partial-Bug: #1524745
This commit is contained in:
Jim Rollenhagen 2017-01-19 20:42:25 +00:00
parent 3c45f2fd1b
commit d46ab0d0b2
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ironic/common/driver_factory.py
View File

@ -347,14 +347,12 @@ def enabled_supported_interfaces(hardware_type):
mapping = dict() mapping = dict()
for interface_type in driver_base.ALL_INTERFACES: for interface_type in driver_base.ALL_INTERFACES:
supported = set() supported = set()
enabled = set()
supported_ifaces = getattr(hardware_type, supported_ifaces = getattr(hardware_type,
'supported_%s_interfaces' % interface_type) 'supported_%s_interfaces' % interface_type)
for name, iface in interfaces(interface_type).items(): for name, iface in interfaces(interface_type).items():
enabled.add(name)
if iface.__class__ in supported_ifaces: if iface.__class__ in supported_ifaces:
supported.add(name) supported.add(name)
mapping[interface_type] = enabled.intersection(supported) mapping[interface_type] = supported
return mapping return mapping