openstack/neutron
Code Issues Proposed changes

Merge "Call install_ingress_direct_goto_flows() when ovs restarts"

Browse Source
This commit is contained in:
Zuul 2021-03-25 21:49:48 +00:00 committed by Gerrit Code Review
commit bfd59b63f3
2 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py
View File

@ -2522,6 +2522,7 @@ class OVSNeutronAgent(l2population_rpc.L2populationRpcCallBackTunnelMixin,
def _handle_ovs_restart(self, polling_manager): def _handle_ovs_restart(self, polling_manager):
self.setup_integration_br() self.setup_integration_br()
self.install_ingress_direct_goto_flows()
self.setup_physical_bridges(self.bridge_mappings) self.setup_physical_bridges(self.bridge_mappings)
if self.enable_tunneling: if self.enable_tunneling:
self._reset_tunnel_ofports() self._reset_tunnel_ofports()

17
neutron/tests/unit/plugins/ml2/drivers/openvswitch/agent/test_ovs_neutron_agent.py
View File

@ -2327,6 +2327,23 @@ class TestOvsNeutronAgent(object):
self._test_ovs_status(constants.OVS_NORMAL, self._test_ovs_status(constants.OVS_NORMAL,
constants.OVS_RESTARTED) constants.OVS_RESTARTED)
def test_ovs_restart_for_ingress_direct_goto_flows(self):
with mock.patch.object(self.agent,
'check_ovs_status',
return_value=constants.OVS_RESTARTED), \
mock.patch.object(self.agent,
'_agent_has_updates',
side_effect=TypeError('loop exit')), \
mock.patch.object(self.agent, 'setup_integration_br'), \
mock.patch.object(self.agent,
'install_ingress_direct_goto_flows') as \
install_ingress_direct_goto_flows:
try:
self.agent.rpc_loop(polling_manager=mock.Mock())
except TypeError:
pass
install_ingress_direct_goto_flows.assert_called_once_with()
def test_rpc_loop_fail_to_process_network_ports_keep_flows(self): def test_rpc_loop_fail_to_process_network_ports_keep_flows(self):
with mock.patch.object(async_process.AsyncProcess, "_spawn"),\ with mock.patch.object(async_process.AsyncProcess, "_spawn"),\
mock.patch.object(async_process.AsyncProcess, "start"),\ mock.patch.object(async_process.AsyncProcess, "start"),\