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/tests/functional | |
| 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/tests/functional')
| -rw-r--r-- | cinderclient/tests/functional/test_cli.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/cinderclient/tests/functional/test_cli.py b/cinderclient/tests/functional/test_cli.py index 01083c4..8047459 100644 --- a/cinderclient/tests/functional/test_cli.py +++ b/cinderclient/tests/functional/test_cli.py @@ -17,7 +17,8 @@ from cinderclient.tests.functional import base class CinderVolumeTests(base.ClientTestBase): """Check of base cinder volume commands.""" - VOLUME_PROPERTY = ('attachments', 'availability_zone', 'bootable', + VOLUME_PROPERTY = ('attachment_ids', 'attached_servers', + 'availability_zone', 'bootable', 'created_at', 'description', 'encrypted', 'id', 'metadata', 'name', 'size', 'status', 'user_id', 'volume_type') |