]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Remove unused 'deprecated' option from auth_strategy opt
authorJay S Bryant <jsbryant@us.ibm.com>
Fri, 2 Oct 2015 17:44:13 +0000 (12:44 -0500)
committerJay S Bryant <jsbryant@us.ibm.com>
Fri, 2 Oct 2015 17:48:17 +0000 (12:48 -0500)
Noticed that the auth_strategy option in
cinder/common/config.py has a 'deprecated' option.  I
can only see that 'no_auth' and 'keystone' are used anywhere
in the code.  Looking at the git logs it appears that
the 'deprecated' option may have been pulled over when we split
from Nova.  This patch removes the unused option.

DocImpact

Change-Id: I34383f9ceb87d10030e1c28f4bc0a7d8c21467d4

cinder/common/config.py

index f7314f70d97443e5927a361c54120beae4e4b7f5..de45905fa70a53397161a53f16040423df088101 100644 (file)
@@ -175,9 +175,8 @@ global_opts = [
                help='The full class name of the volume backup API class'),
     cfg.StrOpt('auth_strategy',
                default='keystone',
-               choices=['noauth', 'keystone', 'deprecated'],
-               help='The strategy to use for auth. Supports noauth, keystone, '
-                    'and deprecated.'),
+               choices=['noauth', 'keystone'],
+               help='The strategy to use for auth. Supports noauth or keystone.'),
     cfg.ListOpt('enabled_backends',
                 default=None,
                 help='A list of backend names to use. These backend names '