From ce67a601b0aa722b66d9559cbca4e34b46494ced Mon Sep 17 00:00:00 2001 From: Gregory Haynes Date: Wed, 16 Sep 2015 16:31:23 -0700 Subject: [PATCH] Use option dest rather than name in missing error 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 --- keystoneauth1/exceptions/auth_plugins.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/keystoneauth1/exceptions/auth_plugins.py b/keystoneauth1/exceptions/auth_plugins.py index 5ad60859..8d129906 100644 --- a/keystoneauth1/exceptions/auth_plugins.py +++ b/keystoneauth1/exceptions/auth_plugins.py @@ -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)