]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
lb: stop handling Havana device updates
authorIhar Hrachyshka <ihrachys@redhat.com>
Fri, 19 Jun 2015 09:34:12 +0000 (11:34 +0200)
committerIhar Hrachyshka <ihrachys@redhat.com>
Tue, 18 Aug 2015 10:48:46 +0000 (12:48 +0200)
In Havana, device details did not contain segmentation_id but vlan_id.
Those times are long gone.

Change-Id: I7ea5d2cb114371692439c0518ce6bf497837e485

neutron/plugins/ml2/drivers/linuxbridge/agent/common/constants.py
neutron/plugins/ml2/drivers/linuxbridge/agent/linuxbridge_neutron_agent.py

index 4e5f65146cc4dfa1024288d9e375a2d5f8a49e23..aa970af2e474582b4f2a115d44a34266d1efb437 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-
-from neutron.plugins.common import constants as p_const
-
-
 FLAT_VLAN_ID = -1
 LOCAL_VLAN_ID = -2
 
@@ -23,16 +19,3 @@ LOCAL_VLAN_ID = -2
 VXLAN_NONE = 'not_supported'
 VXLAN_MCAST = 'multicast_flooding'
 VXLAN_UCAST = 'unicast_flooding'
-
-
-# TODO(rkukura): Eventually remove this function, which provides
-# temporary backward compatibility with pre-Havana RPC and DB vlan_id
-# encoding.
-def interpret_vlan_id(vlan_id):
-    """Return (network_type, segmentation_id) tuple for encoded vlan_id."""
-    if vlan_id == LOCAL_VLAN_ID:
-        return (p_const.TYPE_LOCAL, None)
-    elif vlan_id == FLAT_VLAN_ID:
-        return (p_const.TYPE_FLAT, None)
-    else:
-        return (p_const.TYPE_VLAN, vlan_id)
index 41503b00e8f0f0aa913998d226b8e573ec7a786b..f370b5f9ca280821200603076d67baf71d6c5577 100644 (file)
@@ -935,13 +935,7 @@ class LinuxBridgeNeutronAgentRPC(service.Service):
                 if device_details['admin_state_up']:
                     # create the networking for the port
                     network_type = device_details.get('network_type')
-                    if network_type:
-                        segmentation_id = device_details.get('segmentation_id')
-                    else:
-                        # compatibility with pre-Havana RPC vlan_id encoding
-                        vlan_id = device_details.get('vlan_id')
-                        (network_type,
-                         segmentation_id) = lconst.interpret_vlan_id(vlan_id)
+                    segmentation_id = device_details.get('segmentation_id')
                     if self.br_mgr.add_interface(
                         device_details['network_id'],
                         network_type,