]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Set default core_plugin to None
authorGary Kotton <gkotton@redhat.com>
Mon, 31 Dec 2012 16:03:21 +0000 (16:03 +0000)
committerGary Kotton <gkotton@redhat.com>
Wed, 2 Jan 2013 16:47:54 +0000 (16:47 +0000)
Fixes bug 1093989

Change-Id: Ib0b77994b813e6b2ec0621949371c66d66472896

etc/quantum.conf
quantum/common/config.py
quantum/common/test_lib.py
quantum/manager.py

index 9c837232a43ca54808b1ba0fdbc7e2798ab74c34..a57e5079415f5f5a6cd904978a861906a428bdec 100644 (file)
@@ -38,7 +38,7 @@ bind_port = 9696
 # api_extensions_path =
 
 # Quantum plugin provider module
-core_plugin = quantum.plugins.sample.SamplePlugin.FakePlugin
+# core_plugin =
 
 # Advanced service modules
 # service_plugins =
index 709471e66538e322d34aec5c4dc96a5609e30f29..b634d865fc1817df3fb274ea33909dfaf08d9769 100644 (file)
@@ -40,8 +40,7 @@ core_opts = [
     cfg.StrOpt('api_extensions_path', default=""),
     cfg.StrOpt('policy_file', default="policy.json"),
     cfg.StrOpt('auth_strategy', default='keystone'),
-    cfg.StrOpt('core_plugin',
-               default='quantum.plugins.sample.SamplePlugin.FakePlugin'),
+    cfg.StrOpt('core_plugin'),
     cfg.ListOpt('service_plugins',
                 default=[]),
     cfg.StrOpt('base_mac', default="fa:16:3e:00:00:00"),
index 7e98d5789df525223083f1f118106fe85455ecb0..087fab19bd96d1c1d3b549489bf9cc47c5436164 100644 (file)
@@ -284,7 +284,7 @@ def run_tests(c=None):
 # and override the values in it if needed (e.g., run_tests.py in
 # quantum/plugins/openvswitch/ )
 test_config = {
-    "plugin_name": "quantum.plugins.sample.SamplePlugin.FakePlugin",
+    "plugin_name": "",
     "default_net_op_status": constants.NET_STATUS_ACTIVE,
     "default_port_op_status": constants.PORT_STATUS_ACTIVE,
 }
index 00aca48068ce8aa62e68299f8be9ada4a90d869f..c9053c44ea21d2fd0ad205b44596fa0f1b546cc2 100644 (file)
@@ -71,6 +71,11 @@ class QuantumManager(object):
         if not options:
             options = {}
 
+        if cfg.CONF.core_plugin is None:
+            msg = _('Quantum core_plugin not configured!')
+            LOG.critical(msg)
+            raise Exception(msg)
+
         # NOTE(jkoelker) Testing for the subclass with the __subclasshook__
         #                breaks tach monitoring. It has been removed
         #                intentianally to allow v2 plugins to be monitored