Load all the necessary database tables when running cisco plugin

In Cisco plugin, Remove redundant/unnecessary calls to configure_db().
Make sure that all the database models are loaded before calling
configure_db()

fixes bug #1155121

Change-Id: I27d5dda512140f0553b311b35678c82f729fa854

quantum/plugins/cisco/db/network_db_v2.py

@@ -31,11 +31,6 @@
 LOG = logging.getLogger(__name__)
 
 
-def initialize():
-    """Establish database connection and load models"""
-    db.configure_db()
-
-
 def create_vlanids():
     """Prepopulates the vlan_bindings table"""
     LOG.debug(_("create_vlanids() called"))

quantum/plugins/cisco/db/nexus_db_v2.py

@@ -29,11 +29,6 @@
 LOG = logging.getLogger(__name__)
 
 
-def initialize():
-    """Establish database connection and load models"""
-    db.configure_db()
-
-
 def get_all_nexusport_bindings():
     """Lists all the nexusport bindings"""
     LOG.debug(_("get_all_nexusport_bindings() called"))

quantum/plugins/cisco/models/virt_phy_sw_v2.py

@@ -35,6 +35,7 @@
 from quantum.plugins.cisco import l2network_plugin_configuration as conf
 from quantum.plugins.openvswitch import ovs_db_v2 as odb
 from quantum import quantum_plugin_base_v2
+from quantum.db import api as db_api
 
 
 LOG = logging.getLogger(__name__)
@@ -64,8 +65,6 @@ def __init__(self):
         configured, and load the inventories those device plugins for which the
         inventory is configured
         """
-        cdb.initialize()
-        cred.Store.initialize()
         for key in conf.PLUGINS[const.PLUGINS].keys():
             plugin_obj = conf.PLUGINS[const.PLUGINS][key]
             self._plugins[key] = importutils.import_object(plugin_obj)
@@ -77,12 +76,21 @@ def __init__(self):
                 LOG.debug(_("Loaded device inventory %s\n"),
                           conf.PLUGINS[const.INVENTORY][key])
 
-        if hasattr(self._plugins[const.VSWITCH_PLUGIN],
-                   "supported_extension_aliases"):
+        if ((const.VSWITCH_PLUGIN in self._plugins) and
+            hasattr(self._plugins[const.VSWITCH_PLUGIN],
+                    "supported_extension_aliases")):
             self.supported_extension_aliases.extend(
                 self._plugins[const.VSWITCH_PLUGIN].
                 supported_extension_aliases)
 
+        # At this point, all the database models should have been loaded. It's
+        # possible that configure_db() may have been called by one of the
+        # plugins loaded in above. Otherwise, this call is to make sure that
+        # the database is initialized
+        db_api.configure_db()
+
+        # Initialize credential store after database initialization
+        cred.Store.initialize()
         LOG.debug(_("%(module)s.%(name)s init done"),
                   {'module': __name__,
                    'name': self.__class__.__name__})

quantum/plugins/cisco/network_plugin.py

@@ -66,7 +66,6 @@ def __init__(self):
             self.supported_extension_aliases.extend(
                 self._model.supported_extension_aliases)
 
-        super(PluginV2, self).__init__()
         LOG.debug(_("Plugin initialization complete"))
 
     def __getattribute__(self, name):

quantum/plugins/cisco/nexus/cisco_nexus_plugin_v2.py

@@ -50,8 +50,6 @@ def __init__(self):
         """
         Extracts the configuration parameters from the configuration file
         """
-        # Initialize the nxos db
-        nxos_db.initialize()
         self._client = importutils.import_object(conf.NEXUS_DRIVER)
         LOG.debug(_("Loaded driver %s"), conf.NEXUS_DRIVER)
         self._nexus_switches = conf.NEXUS_DETAILS

quantum/tests/unit/cisco/test_network_plugin.py

@@ -32,12 +32,7 @@ class CiscoNetworkPluginV2TestCase(test_db_plugin.QuantumDbPluginV2TestCase):
     _plugin_name = 'quantum.plugins.cisco.network_plugin.PluginV2'
 
     def setUp(self):
-        def new_init():
-            db.configure_db()
-
-        with mock.patch.object(network_db_v2,
-                               'initialize', new=new_init):
-            super(CiscoNetworkPluginV2TestCase, self).setUp(self._plugin_name)
+        super(CiscoNetworkPluginV2TestCase, self).setUp(self._plugin_name)
         self.port_create_status = 'DOWN'
 
     def _get_plugin_ref(self):