From: Jenkins <jenkins@review.openstack.org>
Date: Thu, 14 May 2015 02:17:30 +0000 (+0000)
Subject: Merge "Refactor checks for device existence"
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=3d71b1ecfa537f6e0d5ffe152ebc65a9c21300dd;p=openstack-build%2Fneutron-build.git

Merge "Refactor checks for device existence"
---

3d71b1ecfa537f6e0d5ffe152ebc65a9c21300dd
diff --cc neutron/agent/l3/router_info.py
index 9b7443cad,3f7684b84..c1336355e
--- a/neutron/agent/l3/router_info.py
+++ b/neutron/agent/l3/router_info.py
@@@ -418,16 -416,21 +416,15 @@@ class RouterInfo(object)
                  for ip in floating_ips]
  
      def _plug_external_gateway(self, ex_gw_port, interface_name, ns_name):
-         if not ip_lib.device_exists(interface_name, namespace=ns_name):
-             self.driver.plug(ex_gw_port['network_id'],
-                              ex_gw_port['id'],
-                              interface_name,
-                              ex_gw_port['mac_address'],
-                              bridge=self.agent_conf.external_network_bridge,
-                              namespace=ns_name,
-                              prefix=EXTERNAL_DEV_PREFIX)
+         self.driver.plug(ex_gw_port['network_id'],
+                          ex_gw_port['id'],
+                          interface_name,
+                          ex_gw_port['mac_address'],
+                          bridge=self.agent_conf.external_network_bridge,
+                          namespace=ns_name,
+                          prefix=EXTERNAL_DEV_PREFIX)
  
 -    def _external_gateway_added(self, ex_gw_port, interface_name,
 -                                ns_name, preserve_ips):
 -        self._plug_external_gateway(ex_gw_port, interface_name, ns_name)
 -
 -        # Build up the interface and gateway IP addresses that
 -        # will be added to the interface.
 -        ip_cidrs = common_utils.fixed_ip_cidrs(ex_gw_port['fixed_ips'])
 +    def _get_external_gw_ips(self, ex_gw_port):
          gateway_ips = []
          enable_ra_on_gw = False
          if 'subnets' in ex_gw_port: