openstack/neutron
Code Issues Proposed changes

Merge "DVR: Correctly change MTU in fip namespace" into stable/queens

Browse Source
This commit is contained in:
Zuul 2019-04-17 03:51:34 +00:00 committed by Gerrit Code Review
commit c9ad47b52c
2 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
neutron/agent/l3/dvr_fip_ns.py
View File

@ -443,13 +443,14 @@ class FipNamespace(namespaces.Namespace):
rtr_2_fip_dev, fip_2_rtr_dev = ip_wrapper.add_veth(rtr_2_fip_name, rtr_2_fip_dev, fip_2_rtr_dev = ip_wrapper.add_veth(rtr_2_fip_name,
fip_2_rtr_name, fip_2_rtr_name,
fip_ns_name) fip_ns_name)
mtu = ri.get_ex_gw_port().get('mtu')
if mtu:
rtr_2_fip_dev.link.set_mtu(mtu)
fip_2_rtr_dev.link.set_mtu(mtu)
rtr_2_fip_dev.link.set_up() rtr_2_fip_dev.link.set_up()
fip_2_rtr_dev.link.set_up() fip_2_rtr_dev.link.set_up()
mtu = ri.get_ex_gw_port().get('mtu')
if mtu:
rtr_2_fip_dev.link.set_mtu(mtu)
fip_2_rtr_dev.link.set_mtu(mtu)
self._add_cidr_to_device(rtr_2_fip_dev, str(rtr_2_fip)) self._add_cidr_to_device(rtr_2_fip_dev, str(rtr_2_fip))
self._add_cidr_to_device(fip_2_rtr_dev, str(fip_2_rtr)) self._add_cidr_to_device(fip_2_rtr_dev, str(fip_2_rtr))

5
neutron/tests/unit/agent/l3/test_dvr_fip_ns.py
View File

@ -302,10 +302,11 @@ class TestDvrFipNs(base.BaseTestCase):
fip_2_rtr_name, fip_2_rtr_name,
fip_ns_name) fip_ns_name)
device.link.set_mtu.assert_called_with(2000)
self.assertEqual(2, device.link.set_mtu.call_count)
self.assertEqual(2, device.link.set_up.call_count) self.assertEqual(2, device.link.set_up.call_count)
device.link.set_mtu.assert_called_with(2000)
self.assertEqual(2, device.link.set_mtu.call_count)
if not addr_exists: if not addr_exists:
expected = [mock.call(str(addr_pair[0]), add_broadcast=False), expected = [mock.call(str(addr_pair[0]), add_broadcast=False),
mock.call(str(addr_pair[1]), add_broadcast=False)] mock.call(str(addr_pair[1]), add_broadcast=False)]