openstack/cinder
Code Issues Proposed changes

Merge "Port infortrend driver to Python 3"

Browse Source
This commit is contained in:
Jenkins 2016-02-24 15:47:46 +00:00 committed by Gerrit Code Review
commit 72c4aae81c
3 changed files with 13 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
cinder/tests/unit/test_infortrend_common.py
View File

@ -1997,6 +1997,10 @@ class InfortrendiSCSICommonTestCase(InfortrendTestCass):
rc, model_update = self.driver.retype(
None, test_volume, test_new_type, test_diff, test_host)
min_size = int(test_volume['size'] * 1024 * 0.2)
create_params = {'init': 'disable', 'min': '%sMB' % min_size}
create_params = ' '.join('%s=%s' % (key, value)
for key, value in create_params.items())
expect_cli_cmd = [
mock.call('ShowSnapshot', 'part=%s' % test_src_part_id),
mock.call(
@ -2004,8 +2008,7 @@ class InfortrendiSCSICommonTestCase(InfortrendTestCass):
fake_pool['pool_id'],
test_volume['id'].replace('-', ''),
'size=%s' % (test_volume['size'] * 1024),
'init=disable min=%sMB' % (
int(test_volume['size'] * 1024 * 0.2))
create_params,
),
mock.call('ShowPartition'),
mock.call(

10
cinder/volume/drivers/infortrend/eonstor_ds_cli/common_cli.py
View File

@ -522,7 +522,7 @@ class InfortrendCommon(object):
if extraspecs_dict:
cmd = self._create_part_parameters_str(extraspecs_dict)
commands = (pool_id, volume_id, 'size=%s' % volume_size, cmd)
commands = (pool_id, volume_id, 'size=%s' % int(volume_size), cmd)
self._execute('CreatePartition', *commands)
def _create_part_parameters_str(self, extraspecs_dict):
@ -776,7 +776,9 @@ class InfortrendCommon(object):
empty_lun_num = 0
min_map_chl = -1
for key, value in self.map_dict[controller].items():
# Sort items to get a reliable behaviour. Dictionary items
# are iterated in a random order because of hash randomization.
for key, value in sorted(self.map_dict[controller].items()):
if empty_lun_num < len(value):
min_map_chl = key
empty_lun_num = len(value)
@ -1303,7 +1305,9 @@ class InfortrendCommon(object):
map_lun = self._get_common_lun_map_id(wwpn_channel_info)
for initiator_wwpn in initiator_target_map:
# Sort items to get a reliable behaviour. Dictionary items
# are iterated in a random order because of hash randomization.
for initiator_wwpn in sorted(initiator_target_map):
for target_wwpn in initiator_target_map[initiator_wwpn]:
channel_id = wwpn_channel_info[target_wwpn.upper()]['channel']
controller = wwpn_channel_info[target_wwpn.upper()]['slot']

1
tests-py3.txt
View File

@ -95,6 +95,7 @@ cinder.tests.unit.test_ibm_xiv_ds8k
cinder.tests.unit.test_ibmnas
cinder.tests.unit.test_image_utils
cinder.tests.unit.test_infortrend_cli
cinder.tests.unit.test_infortrend_common
cinder.tests.unit.test_migrations
cinder.tests.unit.test_misc
cinder.tests.unit.test_netapp