Cleanup auth client path
This commit is contained in:
parent
04730e6f40
commit
9c945bee79
@ -61,22 +61,6 @@ class OpenStackShell(App):
|
|||||||
command_manager=CommandManager('openstack.cli'),
|
command_manager=CommandManager('openstack.cli'),
|
||||||
)
|
)
|
||||||
|
|
||||||
def _authenticate(self, **kwargs):
|
|
||||||
"""Get an auth token from Keystone
|
|
||||||
|
|
||||||
:param username: name of user
|
|
||||||
:param password: user's password
|
|
||||||
:param tenant_id: unique identifier of tenant
|
|
||||||
:param tenant_name: name of tenant
|
|
||||||
:param auth_url: endpoint to authenticate against
|
|
||||||
"""
|
|
||||||
self.ksclient = ksclient.Client(username=kwargs.get('username'),
|
|
||||||
password=kwargs.get('password'),
|
|
||||||
tenant_id=kwargs.get('tenant_id'),
|
|
||||||
tenant_name=kwargs.get('tenant_name'),
|
|
||||||
auth_url=kwargs.get('auth_url'))
|
|
||||||
return self.ksclient.auth_token
|
|
||||||
|
|
||||||
def build_option_parser(self, description, version):
|
def build_option_parser(self, description, version):
|
||||||
parser = super(OpenStackShell, self).build_option_parser(
|
parser = super(OpenStackShell, self).build_option_parser(
|
||||||
description,
|
description,
|
||||||
@ -177,14 +161,23 @@ class OpenStackShell(App):
|
|||||||
'tenant_name': self.options.os_tenant_name,
|
'tenant_name': self.options.os_tenant_name,
|
||||||
'auth_url': self.options.os_auth_url
|
'auth_url': self.options.os_auth_url
|
||||||
}
|
}
|
||||||
token = self._authenticate(**kwargs)
|
self.auth_client = ksclient.Client(
|
||||||
endpoint = self.ksclient.service_catalog.url_for(service_type=cmd.api)
|
username=kwargs.get('username'),
|
||||||
|
password=kwargs.get('password'),
|
||||||
|
tenant_id=kwargs.get('tenant_id'),
|
||||||
|
tenant_name=kwargs.get('tenant_name'),
|
||||||
|
auth_url=kwargs.get('auth_url'),
|
||||||
|
)
|
||||||
|
token = self.auth_client.auth_token
|
||||||
|
endpoint = self.auth_client.service_catalog.url_for(service_type=cmd.api)
|
||||||
|
|
||||||
if self.options.debug:
|
if self.options.debug:
|
||||||
print "api: %s" % cmd.api
|
print "api: %s" % cmd.api
|
||||||
print "token: %s" % token
|
print "token: %s" % token
|
||||||
print "endpoint: %s" % endpoint
|
print "endpoint: %s" % endpoint
|
||||||
|
|
||||||
|
# get a client for the desired api here
|
||||||
|
|
||||||
def clean_up(self, cmd, result, err):
|
def clean_up(self, cmd, result, err):
|
||||||
self.log.debug('clean_up %s', cmd.__class__.__name__)
|
self.log.debug('clean_up %s', cmd.__class__.__name__)
|
||||||
if err:
|
if err:
|
||||||
|
Loading…
Reference in New Issue
Block a user