x/ironic-staging-drivers
Code Issues Proposed changes

Add noop management to iBOOT and WOL hardware types

Browse Source 
The classic drivers had management=None, so no management actions
were tried. The hardware types have management=fake, which does not
support local boot. This ends up in a serious feature regression.

The intended behavior is to use static boot management, which is what
the noop management is about, so use it.

Change-Id: Id46e2747facabda922e1544fa59e90fb31a04a62
This commit is contained in:
Dmitry Tantsur 2018-10-24 15:30:59 +02:00
parent 0dd170b028
commit 58fe0fe457
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ironic_staging_drivers/iboot/__init__.py
View File

@ -16,6 +16,7 @@
from ironic.drivers import generic
from ironic.drivers.modules import fake
from ironic.drivers.modules import noop_mgmt
from ironic_staging_drivers.iboot import power as iboot_power
@ -29,7 +30,7 @@ class IBootHardware(generic.GenericHardware):
@property
def supported_management_interfaces(self):
"""List of supported management interfaces."""
return [fake.FakeManagement]
return [noop_mgmt.NoopManagement, fake.FakeManagement]
@property
def supported_power_interfaces(self):

3
ironic_staging_drivers/wol/__init__.py
View File

@ -15,6 +15,7 @@
from ironic.drivers import generic
from ironic.drivers.modules import fake
from ironic.drivers.modules import noop_mgmt
from ironic_staging_drivers.wol import power as wol_power
@ -28,7 +29,7 @@ class WOLHardware(generic.GenericHardware):
@property
def supported_management_interfaces(self):
"""List of supported management interfaces."""
return [fake.FakeManagement]
return [noop_mgmt.NoopManagement, fake.FakeManagement]
@property
def supported_power_interfaces(self):