]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Modified configuration group name to lowercase
authorSridhar S <sridharas@outlook.com>
Wed, 23 Oct 2013 10:33:40 +0000 (16:03 +0530)
committerSridhar S <sridharas@outlook.com>
Mon, 28 Oct 2013 06:09:38 +0000 (11:39 +0530)
Most of the OpenStack project makes use of lower case for
configuration group names. But, 'neutron' was having group
names in either cases. Bug states it is fine to have group
name in upper case for [DEFAULT] but needs to be changed
for others. It was noted that [PLUMgridDirector] was having
characters.

Change-Id: I95f2dad6d5ae17f722202f4917b6ef460cd41e15
Closes-Bug: #1176683

etc/neutron/plugins/plumgrid/plumgrid.ini
neutron/plugins/plumgrid/plumgrid_plugin/plumgrid_plugin.py

index 2addf617fd83b6cc85026d33b85178e83c48f97d..e9ea3111b7f85ae93e56fc650d4daee1eef7305a 100644 (file)
@@ -1,6 +1,6 @@
 # Config file for Neutron PLUMgrid Plugin
 
-[PLUMgridDirector]
+[plumgriddirector]
 # This line should be pointing to the PLUMgrid Director,
 # for the PLUMgrid platform.
 # director_server=<director-ip-address>
index 2c0b159bc0f9ee96d419c9d8bf7fd23b194b13d3..2eeee405992b987930e24b0efbdcca3766345be9 100644 (file)
@@ -53,7 +53,7 @@ director_server_opts = [
     cfg.IntOpt('servertimeout', default=5,
                help=_("PLUMgrid Director server timeout")), ]
 
-cfg.CONF.register_opts(director_server_opts, "PLUMgridDirector")
+cfg.CONF.register_opts(director_server_opts, "plumgriddirector")
 
 
 class NeutronPluginPLUMgridV2(db_base_plugin_v2.NeutronDbPluginV2,
@@ -80,11 +80,11 @@ class NeutronPluginPLUMgridV2(db_base_plugin_v2.NeutronDbPluginV2,
 
     def plumgrid_init(self):
         """PLUMgrid initialization."""
-        director_plumgrid = cfg.CONF.PLUMgridDirector.director_server
-        director_port = cfg.CONF.PLUMgridDirector.director_server_port
-        director_admin = cfg.CONF.PLUMgridDirector.username
-        director_password = cfg.CONF.PLUMgridDirector.password
-        timeout = cfg.CONF.PLUMgridDirector.servertimeout
+        director_plumgrid = cfg.CONF.plumgriddirector.director_server
+        director_port = cfg.CONF.plumgriddirector.director_server_port
+        director_admin = cfg.CONF.plumgriddirector.username
+        director_password = cfg.CONF.plumgriddirector.password
+        timeout = cfg.CONF.plumgriddirector.servertimeout
 
         # PLUMgrid Director info validation
         LOG.info(_('Neutron PLUMgrid Director: %s'), director_plumgrid)