openstack/python-openstackclient
Code Issues Proposed changes

Merge "Avoid TypeError on message object additions"

Browse Source
This commit is contained in:
Jenkins 2016-05-20 06:37:03 +00:00 committed by Gerrit Code Review
commit a68576b34b
1 changed files with 20 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
openstackclient/api/auth.py
View File

@ -142,14 +142,14 @@ def check_valid_auth_options(options, auth_plugin_name, required_scope=True):
""" """
msg = '' msgs = []
if auth_plugin_name.endswith('password'): if auth_plugin_name.endswith('password'):
if not options.auth.get('username'): if not options.auth.get('username'):
msg += _('Set a username with --os-username, OS_USERNAME,' msgs.append(_('Set a username with --os-username, OS_USERNAME,'
' or auth.username\n') ' or auth.username'))
if not options.auth.get('auth_url'): if not options.auth.get('auth_url'):
msg += _('Set an authentication URL, with --os-auth-url,' msgs.append(_('Set an authentication URL, with --os-auth-url,'
' OS_AUTH_URL or auth.auth_url\n') ' OS_AUTH_URL or auth.auth_url'))
if (required_scope and not if (required_scope and not
options.auth.get('project_id') and not options.auth.get('project_id') and not
options.auth.get('domain_id') and not options.auth.get('domain_id') and not
@ -157,24 +157,28 @@ def check_valid_auth_options(options, auth_plugin_name, required_scope=True):
options.auth.get('project_name') and not options.auth.get('project_name') and not
options.auth.get('tenant_id') and not options.auth.get('tenant_id') and not
options.auth.get('tenant_name')): options.auth.get('tenant_name')):
msg += _('Set a scope, such as a project or domain, set a ' msgs.append(_('Set a scope, such as a project or domain, set a '
'project scope with --os-project-name, OS_PROJECT_NAME ' 'project scope with --os-project-name, '
'or auth.project_name, set a domain scope with ' 'OS_PROJECT_NAME or auth.project_name, set a domain '
'--os-domain-name, OS_DOMAIN_NAME or auth.domain_name') 'scope with --os-domain-name, OS_DOMAIN_NAME or '
'auth.domain_name'))
elif auth_plugin_name.endswith('token'): elif auth_plugin_name.endswith('token'):
if not options.auth.get('token'): if not options.auth.get('token'):
msg += _('Set a token with --os-token, OS_TOKEN or auth.token\n') msgs.append(_('Set a token with --os-token, OS_TOKEN or '
'auth.token'))
if not options.auth.get('auth_url'): if not options.auth.get('auth_url'):
msg += _('Set a service AUTH_URL, with --os-auth-url, ' msgs.append(_('Set a service AUTH_URL, with --os-auth-url, '
'OS_AUTH_URL or auth.auth_url\n') 'OS_AUTH_URL or auth.auth_url'))
elif auth_plugin_name == 'token_endpoint': elif auth_plugin_name == 'token_endpoint':
if not options.auth.get('token'): if not options.auth.get('token'):
msg += _('Set a token with --os-token, OS_TOKEN or auth.token\n') msgs.append(_('Set a token with --os-token, OS_TOKEN or '
'auth.token'))
if not options.auth.get('url'): if not options.auth.get('url'):
msg += _('Set a service URL, with --os-url, OS_URL or auth.url\n') msgs.append(_('Set a service URL, with --os-url, OS_URL or '
'auth.url'))
if msg: if msgs:
raise exc.CommandError('Missing parameter(s): \n%s' % msg) raise exc.CommandError('Missing parameter(s): \n%s' % '\n'.join(msgs))
def build_auth_plugins_option_parser(parser): def build_auth_plugins_option_parser(parser):