From 760870b6c238f5f2aa5fec33852e51b4ebccb718 Mon Sep 17 00:00:00 2001 From: Yang JianFeng Date: Wed, 6 Jun 2018 10:55:00 +0000 Subject: [PATCH] Don't uninstall flow tables in setup_dvr_flows_on_integ_br This is a revise for I7b24a159962af7b58c096a1b2766e2169e9f8aed Br-int's flow tables are already uninstalled in setup_integration_br. And setup_integration_br will install some default flows. If we still uninstall flow tables of br-int in setup_dvr_flows_on_integ_br, these default flow tables will be missing. Closes-Bug: #1775146 Change-Id: I71c1f9034dfc913b9e9ae17cc8f6bd084c9ee580 --- .../drivers/openvswitch/agent/ovs_dvr_neutron_agent.py | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_dvr_neutron_agent.py b/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_dvr_neutron_agent.py index 2e0c6b40738..1c73d5f7c92 100644 --- a/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_dvr_neutron_agent.py +++ b/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_dvr_neutron_agent.py @@ -17,21 +17,16 @@ import sys import netaddr from neutron_lib import constants as n_const -from oslo_config import cfg from oslo_log import log as logging import oslo_messaging from oslo_utils import excutils from osprofiler import profiler -from neutron.agent.common import ovs_lib from neutron.common import utils as n_utils from neutron.plugins.ml2.drivers.openvswitch.agent.common import constants LOG = logging.getLogger(__name__) -cfg.CONF.import_group('AGENT', 'neutron.plugins.ml2.drivers.openvswitch.' - 'agent.common.config') - # A class to represent a DVR-hosted subnet including vif_ports resident on # that subnet @@ -142,7 +137,6 @@ class OVSDVRNeutronAgent(object): self.dvr_mac_address = None if self.enable_distributed_routing: self.get_dvr_mac_address() - self.conf = cfg.CONF def setup_dvr_flows(self): self.setup_dvr_flows_on_integ_br() @@ -209,10 +203,6 @@ class OVSDVRNeutronAgent(object): LOG.info("L2 Agent operating in DVR Mode with MAC %s", self.dvr_mac_address) - # Remove existing flows in integration bridge - if self.conf.AGENT.drop_flows_on_start: - self.int_br.uninstall_flows(cookie=ovs_lib.COOKIE_ANY) - # Add a canary flow to int_br to track OVS restarts self.int_br.setup_canary_table()