]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Correct default service_name for nova_catalog*_info config option
authorTrung Trinh <trung.t.trinh@dektech.com.au>
Mon, 15 Dec 2014 06:23:34 +0000 (20:23 +1400)
committerHuang Zhiteng <winston.d@gmail.com>
Mon, 15 Dec 2014 07:08:09 +0000 (07:08 +0000)
The default service_name in config option (nova_catalog_info and
nova_catalog_admin_info) was mistakenly put as 'nova'. Consequently,
it is impossible to invoke novaclient APIs from Cinder code without
changing the default value of those config options.

The correct service_name defined in Keystone's config file
"/etc/keystone/default_catalog.templates" is:
catalog.RegionOne.compute.name = Compute Service.

This change replaces 'nova' with 'Compute Service' for default value
of 'nova_catalog*_info' config options.

Change-Id: Idee3585a08efa1d6af1f4ee2b74e8cefb05cc54f
Closes-Bug: 1401800

cinder/compute/nova.py

index c58f1055e5a92b0584abc1b7e798b26a99eb65aa..d72a596f9431fd4a049e46dc9ca1e80bf3f77dbe 100644 (file)
@@ -27,13 +27,13 @@ from cinder.openstack.common import log as logging
 
 nova_opts = [
     cfg.StrOpt('nova_catalog_info',
-               default='compute:nova:publicURL',
+               default='compute:Compute Service:publicURL',
                help='Match this value when searching for nova in the '
                     'service catalog. Format is: separated values of '
                     'the form: '
                     '<service_type>:<service_name>:<endpoint_type>'),
     cfg.StrOpt('nova_catalog_admin_info',
-               default='compute:nova:adminURL',
+               default='compute:Compute Service:adminURL',
                help='Same as nova_catalog_info, but for admin endpoint.'),
     cfg.StrOpt('nova_endpoint_template',
                default=None,