]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Plugins should call __init__ of db_base_plugin for db.configure
authorAaron Rosen <aaronorosen@gmail.com>
Wed, 19 Feb 2014 23:08:54 +0000 (15:08 -0800)
committerAaron Rosen <arosen@nicira.com>
Fri, 21 Feb 2014 21:30:12 +0000 (13:30 -0800)
Currently each plugin calls db.configure() within the plugin's __init__
class or defines an initialize() method that's sole job is to call this
method. Instead we should just call the super method of a plugin so that
db.configure() is called for us out of the db_base_plugin class.

Note: the only reason why I'm making this change is that I want to add
something to the __init__() class of the db_base_plugin that's needed for
the nova-event-callback blueprint and adding it in the base class of init
looks to be the best place.

Change-Id: Iec3c912735021ceb90f657108aad3a57460d66e7
Closes-bug: #1282303

30 files changed:
neutron/db/db_base_plugin_v2.py
neutron/plugins/bigswitch/plugin.py
neutron/plugins/brocade/NeutronPlugin.py
neutron/plugins/cisco/db/n1kv_db_v2.py
neutron/plugins/cisco/n1kv/n1kv_neutron_plugin.py
neutron/plugins/linuxbridge/db/l2network_db_v2.py
neutron/plugins/linuxbridge/lb_neutron_plugin.py
neutron/plugins/metaplugin/meta_neutron_plugin.py
neutron/plugins/metaplugin/proxy_neutron_plugin.py
neutron/plugins/midonet/plugin.py
neutron/plugins/ml2/db.py
neutron/plugins/ml2/plugin.py
neutron/plugins/mlnx/db/mlnx_db_v2.py
neutron/plugins/mlnx/mlnx_plugin.py
neutron/plugins/nec/db/api.py
neutron/plugins/nec/nec_plugin.py
neutron/plugins/nicira/NeutronPlugin.py
neutron/plugins/openvswitch/ovs_db_v2.py
neutron/plugins/openvswitch/ovs_neutron_plugin.py
neutron/plugins/plumgrid/plumgrid_plugin/plumgrid_plugin.py
neutron/plugins/ryu/ryu_neutron_plugin.py
neutron/tests/unit/cisco/n1kv/test_n1kv_db.py
neutron/tests/unit/cisco/n1kv/test_n1kv_plugin.py
neutron/tests/unit/linuxbridge/test_lb_db.py
neutron/tests/unit/ml2/test_type_gre.py
neutron/tests/unit/ml2/test_type_vxlan.py
neutron/tests/unit/mlnx/test_mlnx_db.py
neutron/tests/unit/nec/test_ofc_manager.py
neutron/tests/unit/openvswitch/test_ovs_db.py
neutron/tests/unit/test_quota_ext.py

index d6adceef3692922d8e459d9268ffd07ad11d5fa7..33cac83f8941760a5790e583fcfad42a6e6aa656 100644 (file)
@@ -224,10 +224,6 @@ class NeutronDbPluginV2(neutron_plugin_base_v2.NeutronPluginBaseV2,
     __native_sorting_support = True
 
     def __init__(self):
-        # NOTE(jkoelker) This is an incomplete implementation. Subclasses
-        #                must override __init__ and setup the database
-        #                and not call into this class's __init__.
-        #                This connection is setup as memory for the tests.
         db.configure_db()
 
     @classmethod
index d0bc7d46e4b53fcd31a06a05d660337a72de1b50..ef39338580598ae5e181165018a76f18d9b7bb85 100644 (file)
@@ -57,7 +57,6 @@ from neutron.common import topics
 from neutron import context as qcontext
 from neutron.db import agents_db
 from neutron.db import agentschedulers_db
-from neutron.db import api as db
 from neutron.db import db_base_plugin_v2
 from neutron.db import dhcp_rpc_base
 from neutron.db import external_net_db
@@ -102,6 +101,7 @@ class NeutronRestProxyV2Base(db_base_plugin_v2.NeutronDbPluginV2,
     servers = None
 
     def __init__(self, server_timeout=None):
+        super(NeutronRestProxyV2Base, self).__init__()
         # This base class is not intended to be instantiated directly.
         # Extending class should set ServerPool.
         if not self.servers:
@@ -321,11 +321,10 @@ class NeutronRestProxyV2(NeutronRestProxyV2Base,
                                    "dhcp_agent_scheduler", "agent"]
 
     def __init__(self, server_timeout=None):
+        super(NeutronRestProxyV2, self).__init__()
         LOG.info(_('NeutronRestProxy: Starting plugin. Version=%s'),
                  version_string_with_vcs())
         pl_config.register_config()
-        # init DB, proxy's persistent store defaults to in-memory sql-lite DB
-        db.configure_db()
 
         # Include the BigSwitch Extensions path in the api_extensions
         neutron_extensions.append_api_extensions_path(extensions.__path__)
index b49a6bafe17254496f448b9488f090d335022666..f5a04ad205a5a78ed841cbf7fff29bb99b7c3cc8 100644 (file)
@@ -225,6 +225,7 @@ class BrocadePluginV2(db_base_plugin_v2.NeutronDbPluginV2,
         Specify switch address and db configuration.
         """
 
+        super(BrocadePluginV2, self).__init__()
         self.supported_extension_aliases = ["binding", "security-group",
                                             "external-net", "router",
                                             "extraroute", "agent",
@@ -235,7 +236,6 @@ class BrocadePluginV2(db_base_plugin_v2.NeutronDbPluginV2,
                                    physical_interface)
         self.base_binding_dict = self._get_base_binding_dict()
         portbindings_base.register_port_dict_function()
-        db.configure_db()
         self.ctxt = context.get_admin_context()
         self.ctxt.session = db.get_session()
         self._vlan_bitmap = vbm.VlanBitmap(self.ctxt)
index 7c7de737dc5aaba4726413177558a96da4dd849f..01bd40966a332acf870c81d854f3acd7ca1ce601 100644 (file)
@@ -35,11 +35,6 @@ from neutron.plugins.cisco.db import n1kv_models_v2
 LOG = logging.getLogger(__name__)
 
 
-def initialize():
-    """Initialize the database."""
-    db.configure_db()
-
-
 def del_trunk_segment_binding(db_session, trunk_segment_id, segment_pairs):
     """
     Delete a trunk network binding.
index 3c7acb8229983efde65d64b6dd5e5473d4958548..682edd74017a31c8b910e5b386c6f0c8f0e0d280 100644 (file)
@@ -102,7 +102,7 @@ class N1kvNeutronPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
         1. Initialize Nexus1000v and Credential DB
         2. Establish communication with Cisco Nexus1000V
         """
-        n1kv_db_v2.initialize()
+        super(N1kvNeutronPluginV2, self).__init__()
         c_cred.Store.initialize()
         self._initialize_network_ranges()
         self._setup_vsm()
index 786fe841a1f4e87b3f563321b25ab063fc269f97..819d16a3291a090b8527c59beeccc9921f048749 100644 (file)
@@ -29,10 +29,6 @@ from neutron.plugins.linuxbridge.db import l2network_models_v2
 LOG = logging.getLogger(__name__)
 
 
-def initialize():
-    db.configure_db()
-
-
 def sync_network_states(network_vlan_ranges):
     """Synchronize network_states table with current configured VLAN ranges."""
 
index 995b2fe78841bdd334d711bb1598b03b40b75600..c06ce6a76edcd9d3cbb9229a1c9e025fa168ea73 100644 (file)
@@ -252,12 +252,12 @@ class LinuxBridgePluginV2(db_base_plugin_v2.NeutronDbPluginV2,
         return self._aliases
 
     def __init__(self):
+        super(LinuxBridgePluginV2, self).__init__()
         self.base_binding_dict = {
             portbindings.VIF_TYPE: portbindings.VIF_TYPE_BRIDGE,
             portbindings.CAPABILITIES: {
                 portbindings.CAP_PORT_FILTER:
                 'security-group' in self.supported_extension_aliases}}
-        db.initialize()
         self._parse_network_vlan_ranges()
         db.sync_network_states(self.network_vlan_ranges)
         self.tenant_network_type = cfg.CONF.VLANS.tenant_network_type
index fb4ad887768a9415e177f7fe012094f8fdcdb284..1f1a38606e21ec31679322ff0be4671b327ec56e 100644 (file)
@@ -50,6 +50,7 @@ class MetaPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
                    extraroute_db.ExtraRoute_db_mixin):
 
     def __init__(self, configfile=None):
+        super(MetaPluginV2, self).__init__()
         LOG.debug(_("Start initializing metaplugin"))
         self.supported_extension_aliases = ['flavor', 'external-net',
                                             'router', 'ext-gw-mode',
@@ -99,8 +100,6 @@ class MetaPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
             raise exc.Invalid(_('default_l3_flavor %s is not plugin list') %
                               self.default_l3_flavor)
 
-        db.configure_db()
-
         self.extension_map = {}
         if not cfg.CONF.META.extension_map == '':
             extension_list = [method_set.split(':')
index e8faa310d15ee318bd7ead88ab672c3807abe2a4..61cc3402662c176b4c7e69bf3bda4967af32573b 100644 (file)
@@ -17,7 +17,6 @@
 
 from oslo.config import cfg
 
-from neutron.db import api as db
 from neutron.db import db_base_plugin_v2
 from neutron.db import external_net_db
 from neutron.db import l3_db
@@ -35,7 +34,7 @@ class ProxyPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
     supported_extension_aliases = ["external-net", "router"]
 
     def __init__(self, configfile=None):
-        db.configure_db()
+        super(ProxyPluginV2, self).__init__()
         self.neutron = client.Client(
             username=cfg.CONF.PROXY.admin_user,
             password=cfg.CONF.PROXY.admin_password,
index a10b689c1419668ea57e0d31f689aa35da76a8bd..984736dcbc673f986f76c1af2cd7480f88bc2959 100644 (file)
@@ -33,7 +33,6 @@ from neutron.common import rpc as n_rpc
 from neutron.common import topics
 from neutron.db import agents_db
 from neutron.db import agentschedulers_db
-from neutron.db import api as db
 from neutron.db import db_base_plugin_v2
 from neutron.db import dhcp_rpc_base
 from neutron.db import external_net_db
@@ -209,6 +208,7 @@ class MidonetPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
     __native_bulk_support = False
 
     def __init__(self):
+        super(MidonetPluginV2, self).__init__()
         # Read config values
         midonet_conf = cfg.CONF.MIDONET
         midonet_uri = midonet_conf.midonet_uri
@@ -231,7 +231,6 @@ class MidonetPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
             raise MidonetPluginException(msg=msg)
 
         self.setup_rpc()
-        db.configure_db()
 
         self.base_binding_dict = {
             portbindings.VIF_TYPE: portbindings.VIF_TYPE_MIDONET,
index bf911d3d1a2cce497e77925bad045ac32fb98586..0a2d981a5fc075815ea25c978d5485e3955d204d 100644 (file)
@@ -28,10 +28,6 @@ from neutron.plugins.ml2 import models
 LOG = log.getLogger(__name__)
 
 
-def initialize():
-    db_api.configure_db()
-
-
 def add_network_segment(session, network_id, segment):
     with session.begin(subtransactions=True):
         record = models.NetworkSegment(
index 8ca62d7df85725c9ce61a62095fdeb12c480f84a..cacec013e5bb97b13e5a6509d90ceca296980420 100644 (file)
@@ -98,10 +98,10 @@ class Ml2Plugin(db_base_plugin_v2.NeutronDbPluginV2,
         return self._aliases
 
     def __init__(self):
+        super(Ml2Plugin, self).__init__()
         # First load drivers, then initialize DB, then initialize drivers
         self.type_manager = managers.TypeManager()
         self.mechanism_manager = managers.MechanismManager()
-        db.initialize()
         self.type_manager.initialize()
         self.mechanism_manager.initialize()
         # bulk support depends on the underlying drivers
index 50dd7057a42d87cc924f7a567a23a8a1b0b2484b..23f48e2dfa9e223643706520564fe4a473f604fc 100644 (file)
@@ -29,10 +29,6 @@ from neutron.plugins.mlnx.db import mlnx_models_v2
 LOG = logging.getLogger(__name__)
 
 
-def initialize():
-    db.configure_db()
-
-
 def _remove_non_allocatable_vlans(session, allocations,
                                   physical_network, vlan_ids):
     if physical_network in allocations:
index a8cbe90b99a98ddc34a4ddc71fecef9ac9078587..cee2e4b3d8e978ce29935fc018294635df562484 100644 (file)
@@ -95,7 +95,7 @@ class MellanoxEswitchPlugin(db_base_plugin_v2.NeutronDbPluginV2,
 
     def __init__(self):
         """Start Mellanox Neutron Plugin."""
-        db.initialize()
+        super(MellanoxEswitchPlugin, self).__init__()
         self._parse_network_vlan_ranges()
         db.sync_network_states(self.network_vlan_ranges)
         self._set_tenant_network_type()
index e606861ff2247cdb9298fb66b587cd6c247c4c19..b5935f588ff053ab4cfe40e04da080c755fbf010 100644 (file)
@@ -56,10 +56,6 @@ def _get_resource_model(resource, old_style):
         return resource_map[resource]
 
 
-def initialize():
-    db.configure_db()
-
-
 def clear_db(base=model_base.BASEV2):
     db.clear_db(base)
 
index 41aaf61f7c96e473a6f298d148fa4a6dd142a0bf..95e35ffce413c840fb3d4fcfa821693b5d427a6a 100644 (file)
@@ -102,8 +102,7 @@ class NECPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
         return self._aliases
 
     def __init__(self):
-
-        ndb.initialize()
+        super(NECPluginV2, self).__init__()
         self.ofc = ofc_manager.OFCManager()
         self.base_binding_dict = self._get_base_binding_dict()
         portbindings_base.register_port_dict_function()
index c4ad96bea42eef81bb0d1bd23604b480ee31f533..6239ca51fb38359ae0caaf1b0c79836e05c1e6c6 100644 (file)
@@ -38,7 +38,6 @@ from neutron.common import utils
 from neutron import context as q_context
 from neutron.db import agentschedulers_db
 from neutron.db import allowedaddresspairs_db as addr_pair_db
-from neutron.db import api as db
 from neutron.db import db_base_plugin_v2
 from neutron.db import external_net_db
 from neutron.db import extraroute_db
@@ -144,7 +143,7 @@ class NvpPluginV2(addr_pair_db.AllowedAddressPairsMixin,
     novazone_cluster_map = {}
 
     def __init__(self):
-
+        super(NvpPluginV2, self).__init__()
         # TODO(salv-orlando): Replace These dicts with
         # collections.defaultdict for better handling of default values
         # Routines for managing logical ports in NVP
@@ -185,7 +184,6 @@ class NvpPluginV2(addr_pair_db.AllowedAddressPairsMixin,
                 pbin.CAP_PORT_FILTER:
                 'security-group' in self.supported_extension_aliases}}
 
-        db.configure_db()
         self._extend_fault_map()
         self.setup_dhcpmeta_access()
         # Set this flag to false as the default gateway has not
index b28df3e12660455cae9e3b989075a28ecb3111ff..821163d80a41ea9af7f939b9b24bf2a79cb389da 100644 (file)
@@ -33,10 +33,6 @@ from neutron.plugins.openvswitch import ovs_models_v2
 LOG = logging.getLogger(__name__)
 
 
-def initialize():
-    db.configure_db()
-
-
 def get_network_binding(session, network_id):
     session = session or db.get_session()
     try:
index 450ce3dd241230bd2806a7b65c74fc4a754447ea..fde00edeb1e8c89898f209d91d29799438c09f46 100644 (file)
@@ -294,12 +294,12 @@ class OVSNeutronPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
         attributes.NETWORKS, ['_extend_network_dict_provider_ovs'])
 
     def __init__(self, configfile=None):
+        super(OVSNeutronPluginV2, self).__init__()
         self.base_binding_dict = {
             portbindings.VIF_TYPE: portbindings.VIF_TYPE_OVS,
             portbindings.CAPABILITIES: {
                 portbindings.CAP_PORT_FILTER:
                 'security-group' in self.supported_extension_aliases}}
-        ovs_db_v2.initialize()
         self._parse_network_vlan_ranges()
         ovs_db_v2.sync_vlan_allocations(self.network_vlan_ranges)
         self.tenant_network_type = cfg.CONF.OVS.tenant_network_type
index fb389c434b48617f7c088a4305472b6b9db5798e..1f8f99c0d94128b8432c6b0b41f14eea7a7895a9 100644 (file)
@@ -25,7 +25,6 @@ import netaddr
 from oslo.config import cfg
 
 from neutron.api.v2 import attributes
-from neutron.db import api as db
 from neutron.db import db_base_plugin_v2
 from neutron.db import external_net_db
 from neutron.db import l3_db
@@ -69,9 +68,7 @@ class NeutronPluginPLUMgridV2(db_base_plugin_v2.NeutronDbPluginV2,
     def __init__(self):
         LOG.info(_('Neutron PLUMgrid Director: Starting Plugin'))
 
-        # Plugin DB initialization
-        db.configure_db()
-
+        super(NeutronPluginPLUMgridV2, self).__init__()
         self.plumgrid_init()
 
         LOG.debug(_('Neutron PLUMgrid Director: Neutron server with '
index 5fe2845b1f4e93773d081f7dfc29128f507233f6..c81b2069f03f673db8cd37b4df9011661197c433 100644 (file)
@@ -109,13 +109,13 @@ class RyuNeutronPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
         return self._aliases
 
     def __init__(self, configfile=None):
+        super(RyuNeutronPluginV2, self).__init__()
         self.base_binding_dict = {
             portbindings.VIF_TYPE: portbindings.VIF_TYPE_OVS,
             portbindings.CAPABILITIES: {
                 portbindings.CAP_PORT_FILTER:
                 'security-group' in self.supported_extension_aliases}}
         portbindings_base.register_port_dict_function()
-        db.configure_db()
         self.tunnel_key = db_api_v2.TunnelKey(
             cfg.CONF.OVS.tunnel_key_min, cfg.CONF.OVS.tunnel_key_max)
         self.ofp_api_host = cfg.CONF.OVS.openflow_rest_api
index 8e4a0b40f4e71cf4ce815efc6958a601ba238b6e..6339c834ad0992076073bfb9bbe5deeaa2037546 100644 (file)
@@ -100,7 +100,7 @@ class VlanAllocationsTest(base.BaseTestCase):
 
     def setUp(self):
         super(VlanAllocationsTest, self).setUp()
-        n1kv_db_v2.initialize()
+        db.configure_db()
         self.session = db.get_session()
         n1kv_db_v2.sync_vlan_allocations(self.session, VLAN_RANGES)
 
@@ -284,7 +284,7 @@ class VxlanAllocationsTest(base.BaseTestCase,
 
     def setUp(self):
         super(VxlanAllocationsTest, self).setUp()
-        n1kv_db_v2.initialize()
+        db.configure_db()
         self.session = db.get_session()
         n1kv_db_v2.sync_vxlan_allocations(self.session, VXLAN_RANGES)
 
@@ -392,7 +392,7 @@ class NetworkBindingsTest(test_plugin.NeutronDbPluginV2TestCase):
 
     def setUp(self):
         super(NetworkBindingsTest, self).setUp()
-        n1kv_db_v2.initialize()
+        db.configure_db()
         self.session = db.get_session()
 
     def tearDown(self):
@@ -643,7 +643,7 @@ class NetworkProfileTests(base.BaseTestCase,
 
     def setUp(self):
         super(NetworkProfileTests, self).setUp()
-        n1kv_db_v2.initialize()
+        db.configure_db()
         self.session = db.get_session()
 
     def tearDown(self):
@@ -828,7 +828,7 @@ class PolicyProfileTests(base.BaseTestCase):
 
     def setUp(self):
         super(PolicyProfileTests, self).setUp()
-        n1kv_db_v2.initialize()
+        db.configure_db()
         self.session = db.get_session()
 
     def tearDown(self):
@@ -876,7 +876,7 @@ class ProfileBindingTests(base.BaseTestCase,
 
     def setUp(self):
         super(ProfileBindingTests, self).setUp()
-        n1kv_db_v2.initialize()
+        db.configure_db()
         self.session = db.get_session()
 
     def tearDown(self):
index b698e09d8da38670ccd0c960872e79a6f278a995..6cbbeea257dc60ad3b425090744c46231f72c910 100644 (file)
@@ -395,4 +395,4 @@ class TestN1kvNonDbTest(base.BaseTestCase):
 
     """
     def test_db(self):
-        n1kv_db_v2.initialize()
+        db.configure_db()
index 737747c4ebf609722e5e8ebdf7dd55cab4538ace..37885f8141c39c9d48464fa0471000597f489d1e 100644 (file)
@@ -38,7 +38,7 @@ PLUGIN_NAME = ('neutron.plugins.linuxbridge.'
 class NetworkStatesTest(base.BaseTestCase):
     def setUp(self):
         super(NetworkStatesTest, self).setUp()
-        lb_db.initialize()
+        db.configure_db()
         lb_db.sync_network_states(VLAN_RANGES)
         self.session = db.get_session()
         self.addCleanup(db.clear_db)
@@ -154,7 +154,7 @@ class NetworkBindingsTest(test_plugin.NeutronDbPluginV2TestCase):
         cfg.CONF.set_override('network_vlan_ranges', ['physnet1:1000:2999'],
                               group='VLANS')
         super(NetworkBindingsTest, self).setUp(plugin=PLUGIN_NAME)
-        lb_db.initialize()
+        db.configure_db()
         self.session = db.get_session()
 
     def test_add_network_binding(self):
index 4a2b27923039b7bfa58a5c4c5ba883ae4688b501..dd52f0dbf8668fca258bc0d8ae4c0a31cff0fc03 100644 (file)
@@ -18,7 +18,6 @@ from testtools import matchers
 
 from neutron.common import exceptions as exc
 import neutron.db.api as db
-from neutron.plugins.ml2 import db as ml2_db
 from neutron.plugins.ml2 import driver_api as api
 from neutron.plugins.ml2.drivers import type_gre
 from neutron.tests import base
@@ -35,7 +34,7 @@ class GreTypeTest(base.BaseTestCase):
 
     def setUp(self):
         super(GreTypeTest, self).setUp()
-        ml2_db.initialize()
+        db.configure_db()
         self.driver = type_gre.GreTypeDriver()
         self.driver.gre_id_ranges = TUNNEL_RANGES
         self.driver._sync_gre_allocations()
@@ -187,7 +186,7 @@ class GreTypeMultiRangeTest(base.BaseTestCase):
 
     def setUp(self):
         super(GreTypeMultiRangeTest, self).setUp()
-        ml2_db.initialize()
+        db.configure_db()
         self.driver = type_gre.GreTypeDriver()
         self.driver.gre_id_ranges = self.TUNNEL_MULTI_RANGES
         self.driver._sync_gre_allocations()
index e4cd829b3d56339f6feadbf31df0ba9f2d1580d0..f7121126c64dc9f9875bebab899b06b6ab77b0cb 100644 (file)
@@ -21,7 +21,6 @@ from testtools import matchers
 from neutron.common import exceptions as exc
 from neutron.db import api as db
 from neutron.plugins.common import constants as p_const
-from neutron.plugins.ml2 import db as ml2_db
 from neutron.plugins.ml2 import driver_api as api
 from neutron.plugins.ml2.drivers import type_vxlan
 from neutron.tests import base
@@ -42,7 +41,7 @@ VXLAN_UDP_PORT_TWO = 8888
 class VxlanTypeTest(base.BaseTestCase):
     def setUp(self):
         super(VxlanTypeTest, self).setUp()
-        ml2_db.initialize()
+        db.configure_db()
         cfg.CONF.set_override('vni_ranges', [TUNNEL_RANGES],
                               group='ml2_type_vxlan')
         cfg.CONF.set_override('vxlan_group', MULTICAST_GROUP,
@@ -207,7 +206,7 @@ class VxlanTypeMultiRangeTest(base.BaseTestCase):
 
     def setUp(self):
         super(VxlanTypeMultiRangeTest, self).setUp()
-        ml2_db.initialize()
+        db.configure_db()
         self.driver = type_vxlan.VxlanTypeDriver()
         self.driver.vxlan_vni_ranges = self.TUNNEL_MULTI_RANGES
         self.driver._sync_vxlan_allocations()
index b79b871c428f8c218e5e3fd1f061454b1495bb51..c02c19e45088d4a4d3aa1fcd52ad4debf7ad815d 100644 (file)
@@ -35,7 +35,7 @@ TEST_NETWORK_ID = 'abcdefghijklmnopqrstuvwxyz'
 class SegmentationIdAllocationTest(base.BaseTestCase):
     def setUp(self):
         super(SegmentationIdAllocationTest, self).setUp()
-        mlnx_db.initialize()
+        db.configure_db()
         mlnx_db.sync_network_states(VLAN_RANGES)
         self.session = db.get_session()
         self.addCleanup(db.clear_db)
@@ -158,7 +158,7 @@ class SegmentationIdAllocationTest(base.BaseTestCase):
 class NetworkBindingsTest(test_plugin.NeutronDbPluginV2TestCase):
     def setUp(self):
         super(NetworkBindingsTest, self).setUp()
-        mlnx_db.initialize()
+        db.configure_db()
         self.session = db.get_session()
 
     def test_add_network_binding(self):
index 309a4ac3051e9a28c24b1cc8616b52e275e500ac..0d7a5f701053bff1bea1a96227501b9a180c4272 100644 (file)
@@ -18,6 +18,7 @@
 import mock
 
 from neutron import context
+from neutron.db import api as db
 from neutron.openstack.common import uuidutils
 from neutron.plugins.nec.common import config
 from neutron.plugins.nec.db import api as ndb
@@ -44,9 +45,9 @@ class OFCManagerTestBase(base.BaseTestCase):
 
     def setUp(self):
         super(OFCManagerTestBase, self).setUp()
+        db.configure_db()
         driver = "neutron.tests.unit.nec.stub_ofc_driver.StubOFCDriver"
         config.CONF.set_override('driver', driver, 'OFC')
-        ndb.initialize()
         self.addCleanup(ndb.clear_db)
         self.ofc = ofc_manager.OFCManager()
         # NOTE: enable_autocheck() is a feature of StubOFCDriver
index b53fa4df29abe7bd7a07b9b6807075a8c2139644..91aefe09da232a139cb8a1fdabe4bff394bd0d9e 100644 (file)
@@ -48,7 +48,7 @@ PLUGIN_NAME = ('neutron.plugins.openvswitch.'
 class VlanAllocationsTest(base.BaseTestCase):
     def setUp(self):
         super(VlanAllocationsTest, self).setUp()
-        ovs_db_v2.initialize()
+        db.configure_db()
         ovs_db_v2.sync_vlan_allocations(VLAN_RANGES)
         self.session = db.get_session()
         self.addCleanup(db.clear_db)
@@ -192,7 +192,7 @@ class VlanAllocationsTest(base.BaseTestCase):
 class TunnelAllocationsTest(base.BaseTestCase):
     def setUp(self):
         super(TunnelAllocationsTest, self).setUp()
-        ovs_db_v2.initialize()
+        db.configure_db()
         ovs_db_v2.sync_tunnel_allocations(TUNNEL_RANGES)
         self.session = db.get_session()
         self.addCleanup(db.clear_db)
@@ -302,7 +302,7 @@ class NetworkBindingsTest(test_plugin.NeutronDbPluginV2TestCase):
         cfg.CONF.set_override('network_vlan_ranges', ['physnet1:1000:2999'],
                               group='OVS')
         super(NetworkBindingsTest, self).setUp(plugin=PLUGIN_NAME)
-        ovs_db_v2.initialize()
+        db.configure_db()
         self.session = db.get_session()
 
     def test_add_network_binding(self):
index 627e257da1b6a32e4d8e8814fc22cd250388477f..7c395c1331a0394da3444e67c98dd2253ac1cf2e 100644 (file)
@@ -29,7 +29,6 @@ from neutron.common import exceptions
 from neutron import context
 from neutron.db import api as db
 from neutron.db import quota_db
-from neutron.plugins.linuxbridge.db import l2network_db_v2
 from neutron import quota
 from neutron.tests import base
 from neutron.tests.unit import test_api_v2
@@ -73,7 +72,7 @@ class QuotaExtensionTestCase(testlib_api.WebTestCase):
         # extra1 here is added later, so have to do it manually
         quota.QUOTAS.register_resource_by_name('extra1')
         ext_mgr = extensions.PluginAwareExtensionManager.get_instance()
-        l2network_db_v2.initialize()
+        db.configure_db()
         app = config.load_paste_app('extensions_test_app')
         ext_middleware = extensions.ExtensionMiddleware(app, ext_mgr=ext_mgr)
         self.api = webtest.TestApp(ext_middleware)