openstack
/
python-novaclient
Code Issues Proposed changes

Merge "Set the lower limit of api_version for volume_type"

This commit is contained in:
Zuul 2019-05-23 14:38:16 +00:00 committed by Gerrit Code Review
parent bdbd0df064 7b9d300089
commit bdc49fb621
2 changed files with 29 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
novaclient/tests/unit/v2/test_shell.py
View File

@ -624,6 +624,34 @@ class ShellTest(utils.TestCase):
'size=1,format=ext4,type=disk,shutdown=foobar '
'some-server' % FAKE_UUID_1))
def test_boot_from_volume_with_volume_type_latest_microversion(self):
self.run_command(
'boot --flavor 1 --block-device id=%s,source=image,dest=volume,'
'size=1,bootindex=0,shutdown=remove,tag=foo,volume_type=lvm '
'bfv-server' % FAKE_UUID_1, api_version='2.latest')
self.assert_called_anytime(
'POST', '/servers',
{'server': {
'flavorRef': '1',
'name': 'bfv-server',
'block_device_mapping_v2': [
{
'uuid': FAKE_UUID_1,
'source_type': 'image',
'destination_type': 'volume',
'volume_size': '1',
'delete_on_termination': True,
'tag': 'foo',
'boot_index': '0',
'volume_type': 'lvm'
},
],
'networks': 'auto',
'imageRef': '',
'min_count': 1,
'max_count': 1,
}})
def test_boot_from_volume_with_volume_type_old_microversion(self):
ex = self.assertRaises(
exceptions.CommandError, self.run_command,

2
novaclient/v2/shell.py
View File

@ -145,7 +145,7 @@ def _parse_block_device_mapping_v2(cs, args, image):
'delete_on_termination': False}
bdm.append(bdm_dict)
supports_volume_type = cs.api_version == api_versions.APIVersion('2.67')
supports_volume_type = cs.api_version >= api_versions.APIVersion('2.67')
for device_spec in args.block_device:
spec_dict = _parse_device_spec(device_spec)