diff options
| author | whoami-rajat <rajatdhasmana@gmail.com> | 2019-12-05 05:18:38 +0000 |
|---|---|---|
| committer | Rajat Dhasmana <rajatdhasmana@gmail.com> | 2020-02-21 07:37:28 +0000 |
| commit | 8fa0882354c589b17d4502e04344ae00b75369d5 (patch) | |
| tree | 88e5b5f5df9c6362163e7ecf77f0393d53d4cd9b | |
| parent | 523d09192bc7fec557d4fe49b9da678dd68d76d5 (diff) | |
| download | python-cinderclient-8fa0882354c589b17d4502e04344ae00b75369d5.tar.gz | |
Fix: --poll inconsistency5.0.1
When we use `--poll` parameter with cinder create command, it waits
for the volume to become available but doesn't update the data
displayed to the user.
Due to this, there are inconsistency between several parameters
in the output after 'poll' and 'cinder show' command.
Eg: cinder create 1 --image <image-id> --poll
shows 'bootable' flag as false
whereas, cinder show <vol-id>
shows 'bootable' as true
Change-Id: I1502e88f1cd84d225b75c07313e4eb252cc2d645
Closes-Bug: #1855224
(cherry picked from commit ceddb3cfd033661f63d1245a45e0e197561be19c)
| -rw-r--r-- | cinderclient/tests/unit/v3/test_shell.py | 1 | ||||
| -rw-r--r-- | cinderclient/v3/shell.py | 2 |
2 files changed, 2 insertions, 1 deletions
diff --git a/cinderclient/tests/unit/v3/test_shell.py b/cinderclient/tests/unit/v3/test_shell.py index ee35ddf..f76a484 100644 --- a/cinderclient/tests/unit/v3/test_shell.py +++ b/cinderclient/tests/unit/v3/test_shell.py @@ -1165,7 +1165,6 @@ class ShellTest(utils.TestCase): volume = self.shell.cs.volumes.get('1234') info = dict() info.update(volume._info) - info.pop('links', None) self.assertEqual(1, poll_method.call_count) timeout_period = 3600 poll_method.assert_has_calls([mock.call(self.shell.cs.volumes.get, diff --git a/cinderclient/v3/shell.py b/cinderclient/v3/shell.py index fa1cc9c..fcb9e4a 100644 --- a/cinderclient/v3/shell.py +++ b/cinderclient/v3/shell.py @@ -675,6 +675,8 @@ def do_create(cs, args): shell_utils._poll_for_status( cs.volumes.get, volume.id, info, 'creating', ['available'], timeout_period, cs.client.global_request_id, cs.messages) + volume = cs.volumes.get(volume.id) + info.update(volume._info) utils.print_dict(info) |