openstack/keystoneauth
Code Issues Proposed changes

Use option dest rather than name in missing error

Browse Source 
When we are missing a required option we should use option.dest rather
then option.name so a user is presented with the correct value to use
(otherwise we say auth-url is missing when auth_url is what we actually
wanted, for example).

Change-Id: Ie1d72f6969bfcebdf94619158eb94ccac3bc75ba
This commit is contained in:
Gregory Haynes 2015-09-16 16:31:23 -07:00
parent 7d2d29aedd
commit ce67a601b0
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
keystoneauth1/exceptions/auth_plugins.py
View File

@ -89,6 +89,6 @@ class MissingRequiredOptions(OptionError):
def __init__(self, options):
self.options = options
names = ", ".join(o.name for o in options)
names = ", ".join(o.dest for o in options)
m = 'Auth plugin requires parameters which were not given: %s'
super(MissingRequiredOptions, self).__init__(m % names)