openstack/ironic
Code Issues Proposed changes

Merge "log if 'flat' interface and no cleaning network"

Browse Source
This commit is contained in:
Jenkins
2017-02-07 20:46:14 +00:00
committed by Gerrit Code Review
parent 70a6301115 e3bd99f93a
commit 9da91aaaf5
2 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
ironic/drivers/modules/network/flat.py
View File

@@ -36,16 +36,12 @@ class FlatNetwork(common.VIFPortIDMixin, neutron.NeutronNetworkInterfaceMixin,
def __init__(self):
cleaning_net = CONF.neutron.cleaning_network
# TODO(vdrok): Switch to DriverLoadError in Ocata
if not cleaning_net:
LOG.warning(_LW(
'Please specify a valid UUID or name for '
'[neutron]/cleaning_network configuration option so that '
'this interface is able to perform cleaning. It will be '
'required starting with the Ocata release, and if not '
'specified then, the conductor service will fail to start if '
'"flat" is in the list of values for '
'[DEFAULT]enabled_network_interfaces configuration option.'))
'this interface is able to perform cleaning. Otherwise, '
'cleaning operations will fail to start.'))
def validate(self, task):
"""Validates the network interface.

2
ironic/tests/unit/drivers/modules/network/test_flat.py
View File

@@ -68,7 +68,7 @@ class TestFlatInterface(db_base.DbTestCase):
mock_p_changed.assert_called_once_with(task, port)
@mock.patch.object(flat_interface, 'LOG')
def test_init_incorrect_cleaning_net(self, mock_log):
def test_init_no_cleaning_network(self, mock_log):
self.config(cleaning_network=None, group='neutron')
flat_interface.FlatNetwork()
self.assertTrue(mock_log.warning.called)