]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
ml2: remove stale _filter_nets_l3 in get_networks
authorIsaku Yamahata <isaku.yamahata@intel.com>
Fri, 17 Oct 2014 06:35:44 +0000 (15:35 +0900)
committerIsaku Yamahata <isaku.yamahata@intel.com>
Tue, 20 Jan 2015 23:13:20 +0000 (15:13 -0800)
The commit of 0156ec175cc047826b211727d43d5d14a3e1f2d2
change-id of I47e01a11afaf6e6bcf06da7bd713fd39b05600ff
which fixes bug 1132849 removed the call of _filter_nets_l3 methods.
But somehow the fix missed ml2 plugin.
This patch fixes ml2 plugin and removes the unused mothod.

Change-Id: I4d13223c170fd6777773970e0d22a191b98dd5ee
Closes-Bug: #1382360

neutron/db/external_net_db.py
neutron/plugins/ml2/plugin.py

index 313d97b56d72cffa724f3efab519fb245757efea..178068e14879d658c3996b801e6a0a7c2e2cf98f 100644 (file)
@@ -144,18 +144,6 @@ class External_net_db_mixin(object):
         if l3plugin:
             l3plugin.delete_disassociated_floatingips(context, network_id)
 
-    def _filter_nets_l3(self, context, nets, filters):
-        vals = filters and filters.get(external_net.EXTERNAL, [])
-        if not vals:
-            return nets
-
-        ext_nets = set(en['network_id']
-                       for en in context.session.query(ExternalNetwork))
-        if vals[0]:
-            return [n for n in nets if n['id'] in ext_nets]
-        else:
-            return [n for n in nets if n['id'] not in ext_nets]
-
     def get_external_network_id(self, context):
         nets = self.get_networks(context, {external_net.EXTERNAL: [True]})
         if len(nets) > 1:
index d58c8074a91dd8145a814e4bc3c71127f52b439b..98b1d56c54a68587ed196d8e2dfb3d8010345ccb 100644 (file)
@@ -615,7 +615,6 @@ class Ml2Plugin(db_base_plugin_v2.NeutronDbPluginV2,
                 self.type_manager._extend_network_dict_provider(context, net)
 
             nets = self._filter_nets_provider(context, nets, filters)
-            nets = self._filter_nets_l3(context, nets, filters)
 
         return [self._fields(net, fields) for net in nets]