diff --git a/cinder/tests/unit/test_infortrend_common.py b/cinder/tests/unit/test_infortrend_common.py index 4045a433b..1b0d15273 100644 --- a/cinder/tests/unit/test_infortrend_common.py +++ b/cinder/tests/unit/test_infortrend_common.py @@ -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( diff --git a/cinder/volume/drivers/infortrend/eonstor_ds_cli/common_cli.py b/cinder/volume/drivers/infortrend/eonstor_ds_cli/common_cli.py index a38f863de..c3c6cb42b 100644 --- a/cinder/volume/drivers/infortrend/eonstor_ds_cli/common_cli.py +++ b/cinder/volume/drivers/infortrend/eonstor_ds_cli/common_cli.py @@ -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'] diff --git a/tests-py3.txt b/tests-py3.txt index e47dd0f82..86f536220 100644 --- a/tests-py3.txt +++ b/tests-py3.txt @@ -116,6 +116,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