openstack/neutron
Code Issues Proposed changes

Merge "DhcpLocalProcess._enable should not call DhcpLocalProcess.restart"

Browse Source
This commit is contained in:
Zuul 2020-01-22 07:01:03 +00:00 committed by Gerrit Code Review
commit 5fb92c16e4
2 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
neutron/agent/linux/dhcp.py
View File

@ -261,8 +261,9 @@ class DhcpLocalProcess(DhcpBase):
def _enable(self):
try:
if self.active:
self.restart()
elif self._enable_dhcp():
self.disable(retain_port=True, block=True)
if self._enable_dhcp():
fileutils.ensure_tree(self.network_conf_dir, mode=0o755)
interface_name = self.device_manager.setup(self.network)
self.interface_name = interface_name

12
neutron/tests/unit/agent/linux/test_dhcp.py
View File

@ -1019,9 +1019,6 @@ class LocalChild(dhcp.DhcpLocalProcess):
def reload_allocations(self):
self.called.append('reload')
def restart(self):
self.called.append('restart')
def spawn_process(self):
self.called.append('spawn')
@ -1120,12 +1117,13 @@ class TestDhcpLocalProcess(TestBase):
def test_enable_already_active(self):
with mock.patch.object(LocalChild, 'active') as patched:
patched.__get__ = mock.Mock(return_value=True)
patched.__get__ = mock.Mock(side_effect=[True, False])
lp = LocalChild(self.conf, FakeV4Network())
lp.enable()
with mock.patch.object(ip_lib, 'delete_network_namespace'):
lp.enable()
self.assertEqual(lp.called, ['restart'])
self.assertFalse(self.mock_mgr.return_value.setup.called)
self.assertEqual(lp.called, ['spawn'])
self.assertTrue(self.mock_mgr.return_value.setup.called)
@mock.patch.object(fileutils, 'ensure_tree')
def test_enable(self, ensure_dir):