diff --git a/tripleo_common/tests/test_upgrade.py b/tripleo_common/tests/test_upgrade.py index 1a9fb2f1f..99509dcff 100644 --- a/tripleo_common/tests/test_upgrade.py +++ b/tripleo_common/tests/test_upgrade.py @@ -80,31 +80,3 @@ class StackUpgradeManagerTest(base.TestCase): heatclient.stacks.update.assert_called_once_with(**params) mock_env_files.assert_called_once_with(env_paths=[ '/tmp/environments/major-upgrade-pacemaker-converge.yaml']) - - @mock.patch('heatclient.common.template_utils.' - 'process_multiple_environments_and_files') - @mock.patch('heatclient.common.template_utils.get_template_contents') - def test_upgrade_pre(self, mock_template_contents, mock_env_files): - heatclient = mock.MagicMock() - heatclient.stacks.get.return_value = mock.MagicMock( - stack_name='stack', id='stack_id') - mock_template_contents.return_value = ({}, 'template body') - mock_env_files.return_value = ({}, {}) - upgrade.StackUpgradeManager( - heatclient=heatclient, - stack_id='stack_id', - tht_dir='/tmp/' - ).upgrade_pre() - params = { - 'existing': True, - 'stack_id': 'stack_id', - 'template': 'template body', - 'files': {}, - 'parameters': {}, - 'environment': { - }, - 'timeout_mins': 240, - } - heatclient.stacks.update.assert_called_once_with(**params) - mock_env_files.assert_called_once_with(env_paths=[ - '/tmp/environments/major-upgrade-pacemaker-init.yaml']) diff --git a/tripleo_common/upgrade.py b/tripleo_common/upgrade.py index ad284521a..60ead6412 100644 --- a/tripleo_common/upgrade.py +++ b/tripleo_common/upgrade.py @@ -22,7 +22,7 @@ from tripleo_common import constants LOG = logging.getLogger(__name__) -UPGRADE_PREPARE_ENVIRONMENT_NAME = 'major-upgrade-pacemaker-init.yaml' +TEMPLATE_NAME = 'overcloud-without-mergepy.yaml' UPGRADE_ENVIRONMENT_NAME = 'major-upgrade-pacemaker.yaml' UPGRADE_CLEANUP_ENVIRONMENT_NAME = 'major-upgrade-pacemaker-converge.yaml' @@ -63,10 +63,6 @@ class StackUpgradeManager(object): self.heatclient.stacks.update(**fields) - def upgrade_pre(self, timeout_mins=constants.STACK_TIMEOUT_DEFAULT): - LOG.info('upgrading stack: %s', self.stack.stack_name) - self._update_stack(timeout_mins, UPGRADE_PREPARE_ENVIRONMENT_NAME) - def upgrade(self, timeout_mins=constants.STACK_TIMEOUT_DEFAULT): LOG.info('upgrading stack: %s', self.stack.stack_name) self._update_stack(timeout_mins, UPGRADE_ENVIRONMENT_NAME)