x/python-tatuclient
Code Issues Proposed changes

Show certificates only on single records, not lists.

Browse Source
This commit is contained in:
Pino de Candia 2018-01-30 06:02:50 +00:00
parent ea431b2011
commit f29cf34fb8
5 changed files with 9 additions and 42 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
setup.cfg
View File

@ -39,18 +39,14 @@ openstack.ssh.v1 =
usercert_create = tatuclient.v1.cli.usercert:CreateUserCertCommand
usercert_list = tatuclient.v1.cli.usercert:ListUserCertCommand
usercert_show = tatuclient.v1.cli.usercert:ShowUserCertCommand
usercert_cert_show = tatuclient.v1.cli.usercert:ShowUserCertCertCommand
usercert_revoke = tatuclient.v1.cli.usercert:RevokeUserCertCommand
hostcert_list = tatuclient.v1.cli.hostcert:ListHostCertCommand
hostcert_show = tatuclient.v1.cli.hostcert:ShowHostCertCommand
hostcert_cert_show = tatuclient.v1.cli.hostcert:ShowHostCertCertCommand
sshca_create = tatuclient.v1.cli.ca:CreateCACommand
sshca_list = tatuclient.v1.cli.ca:ListCACommand
sshca_show = tatuclient.v1.cli.ca:ShowCACommand
sshca_user_key_show = tatuclient.v1.cli.ca:ShowCAUserKeyCommand
sshca_host_key_show = tatuclient.v1.cli.ca:ShowCAHostKeyCommand
[build_sphinx]
builders = html,man

2
tatuclient/utils.py
View File

@ -69,7 +69,7 @@ def get_item_properties(item, fields, mixed_case_fields=[], formatters={}):
if field in formatters:
row.append(formatters[field](item))
else:
row.append(get_property(item, field, mixed_case_fields))
row.append(get_item_property(item, field, mixed_case_fields))
return tuple(row)

16
tatuclient/v1/cli/ca.py
View File

@ -63,22 +63,6 @@ class ShowCACommand(command.ShowOne):
return _names, utils.get_item_properties(data, _columns)
class ShowCAUserKeyCommand(ShowCACommand):
"""Print the CA's unformatted public key for user certificates."""
def take_action(self, parsed_args):
data = self._get_data(parsed_args)
self.app.stdout.write(utils.get_item_property(data, 'user_pub_key'))
class ShowCAHostKeyCommand(ShowCACommand):
"""Print the CA's unformatted public key for user certificates."""
def take_action(self, parsed_args):
data = self._get_data(parsed_args)
self.app.stdout.write(utils.get_item_property(data, 'host_pub_key'))
class CreateCACommand(command.ShowOne):
"""Create new CA"""

15
tatuclient/v1/cli/hostcert.py
View File

@ -24,8 +24,8 @@ from tatuclient.v1.utils import get_all
LOG = logging.getLogger(__name__)
_columns = ['host_id', 'srv_url', 'pat_bastions', 'fingerprint']
_names = ['Instance ID', 'SRV URL', 'PAT Bastions', 'Fingerprint']
_columns = ['host_id', 'hostname', 'srv_url', 'pat_bastions', 'fingerprint']
_names = ['Instance ID', 'Hostname', 'SRV URL', 'PAT Bastions', 'Fingerprint']
class ListHostCertCommand(command.Lister):
@ -60,12 +60,5 @@ class ShowHostCertCommand(command.ShowOne):
def take_action(self, parsed_args):
data = self._get_data(parsed_args)
return _names, utils.get_item_properties(data, _columns)
class ShowUserCertCertCommand(ShowUserCertCommand):
"""Print the HostCert's unformatted certificate data."""
def take_action(self, parsed_args):
data = self._get_data(parsed_args)
self.app.stdout.write(utils.get_item_property(data, 'cert'))
return (_names + ['Certificate'],
utils.get_item_properties(data, _columns + ['cert']))

14
tatuclient/v1/cli/usercert.py
View File

@ -59,15 +59,8 @@ class ShowUserCertCommand(command.ShowOne):
def take_action(self, parsed_args):
data = self._get_data(parsed_args)
return _names, utils.get_item_properties(data, _columns)
class ShowUserCertCertCommand(ShowUserCertCommand):
"""Print the UserCert's unformatted certificate data."""
def take_action(self, parsed_args):
data = self._get_data(parsed_args)
self.app.stdout.write(utils.get_item_property(data, 'cert'))
return (_names + ['Certificate'],
utils.get_item_properties(data, _columns + ['cert']))
class CreateUserCertCommand(command.ShowOne):
@ -85,7 +78,8 @@ class CreateUserCertCommand(command.ShowOne):
data = client.usercert.create(client.session.get_user_id(),
client.session.get_project_id(),
parsed_args.pub_key)
return _names, utils.get_item_properties(data, _columns)
return (_names + ['Certificate'],
utils.get_item_properties(data, _columns + ['cert']))
class RevokeUserCertCommand(command.ShowOne):