diff options
| author | John Griffith <john.griffith8@gmail.com> | 2017-06-22 20:35:24 +0000 |
|---|---|---|
| committer | John Griffith <john.griffith8@gmail.com> | 2017-07-03 18:06:38 +0000 |
| commit | 0ea7b2e0cf1dfd31ee8f12ceae5193c4a68c60b3 (patch) | |
| tree | 583e498323ea051e023f2fe24dc650199261f995 /cinderclient/v2/shell.py | |
| parent | 44e650a4ed183af9fa977af192355450afd4f1b7 (diff) | |
| download | python-cinderclient-0ea7b2e0cf1dfd31ee8f12ceae5193c4a68c60b3.tar.gz | |
cinder show with attachments is a mess
The show command currently just dumps the entire attachments
dictionary into the output which is a real mess and completely
screws up the displayed output (shell command).
There's really no reason to do this, we can just give the
attachment ID's and then when you're on the newer versions
you can do an attachment-show for all the crazy details if
you want. Keep in mind that the list command already shows
the server-id we're attached too, but that might also be
nice from the show command rather than jumping through
multiple commands. To try and accomodate various use cases
we'll also add an "attached_servers" field to the show command,
but you'll have to coorelate manually from there.
Change-Id: I45ac49d8d9a185c52727c5bc24a6a1323be83689
Closes-Bug: #1494941
Diffstat (limited to 'cinderclient/v2/shell.py')
| -rw-r--r-- | cinderclient/v2/shell.py | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/cinderclient/v2/shell.py b/cinderclient/v2/shell.py index 7186fe8..9777378 100644 --- a/cinderclient/v2/shell.py +++ b/cinderclient/v2/shell.py @@ -32,6 +32,18 @@ from cinderclient import utils from cinderclient.v2 import availability_zones +def _translate_attachments(info): + attachments = [] + attached_servers = [] + for attachment in info['attachments']: + attachments.append(attachment['id']) + attached_servers.append(attachment['server_id']) + info.pop('attachments', None) + info['attachment_ids'] = attachments + info['attached_servers'] = attached_servers + return info + + @utils.arg('--all-tenants', dest='all_tenants', metavar='<0|1>', @@ -188,9 +200,10 @@ def do_show(cs, args): info['readonly'] = info['metadata']['readonly'] info.pop('links', None) + info = _translate_attachments(info) utils.print_dict(info, formatters=['metadata', 'volume_image_metadata', - 'attachments']) + 'attachment_ids', 'attached_servers']) class CheckSizeArgForCreate(argparse.Action): @@ -340,6 +353,7 @@ def do_create(cs, args): info['readonly'] = info['metadata']['readonly'] info.pop('links', None) + info = _translate_attachments(info) utils.print_dict(info) |