diff options
| author | Jenkins <jenkins@review.openstack.org> | 2017-04-03 19:49:42 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2017-04-03 19:49:42 +0000 |
| commit | e2cd971a03ed8b8bf518b995d32c49a85cae441d (patch) | |
| tree | 18b0c683601af22b8436a4600fe1db81cc37a64c | |
| parent | 6251d7d82a0c4be5c6e82d5db5d26e5d95cb4c96 (diff) | |
| parent | 61cde9c8e85182073b53e3736f8568f2a6d1453c (diff) | |
| download | python-openstackclient-e2cd971a03ed8b8bf518b995d32c49a85cae441d.tar.gz | |
Merge "Fix block-device-mapping when volume_size is empty"
| -rw-r--r-- | openstackclient/compute/v2/server.py | 2 | ||||
| -rw-r--r-- | openstackclient/tests/unit/compute/v2/test_server.py | 3 | ||||
| -rw-r--r-- | releasenotes/notes/bug-1677236-7de9d11c3f0fb5ed.yaml | 6 |
3 files changed, 8 insertions, 3 deletions
diff --git a/openstackclient/compute/v2/server.py b/openstackclient/compute/v2/server.py index 1fe5bb0d..3ae75167 100644 --- a/openstackclient/compute/v2/server.py +++ b/openstackclient/compute/v2/server.py @@ -544,7 +544,7 @@ class CreateServer(command.ShowOne): else: mapping['source_type'] = 'volume' mapping['destination_type'] = 'volume' - if len(dev_map) > 2: + if len(dev_map) > 2 and dev_map[2]: mapping['volume_size'] = dev_map[2] if len(dev_map) > 3: mapping['delete_on_termination'] = dev_map[3] diff --git a/openstackclient/tests/unit/compute/v2/test_server.py b/openstackclient/tests/unit/compute/v2/test_server.py index 7691ef59..a0716e4c 100644 --- a/openstackclient/tests/unit/compute/v2/test_server.py +++ b/openstackclient/tests/unit/compute/v2/test_server.py @@ -817,8 +817,7 @@ class TestServerCreate(TestServer): 'uuid': real_volume_mapping.split(':', 1)[0], 'destination_type': 'volume', 'source_type': 'volume', - 'delete_on_termination': '0', - 'volume_size': '' + 'delete_on_termination': '0' }], nics=[], scheduler_hints={}, diff --git a/releasenotes/notes/bug-1677236-7de9d11c3f0fb5ed.yaml b/releasenotes/notes/bug-1677236-7de9d11c3f0fb5ed.yaml new file mode 100644 index 00000000..d67cc5c9 --- /dev/null +++ b/releasenotes/notes/bug-1677236-7de9d11c3f0fb5ed.yaml @@ -0,0 +1,6 @@ +--- +fixes: + - | + Fix creating a server with a block-device-mapping when volume_size + is empty. + [Bug `1677236 <https://bugs.launchpad.net/bugs/1652827>`_] |