AMBARI-21709 Finalize Warns that it is Permanent Even For PATCH Upgrades. (atkach)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/2a57959f Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/2a57959f Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/2a57959f Branch: refs/heads/feature-branch-AMBARI-21307 Commit: 2a57959f5b8ddaa545dba1b5496f82902fa0e75c Parents: b4f0c5a Author: Andrii Tkach <[email protected]> Authored: Fri Aug 11 16:48:47 2017 +0300 Committer: Andrii Tkach <[email protected]> Committed: Mon Aug 28 13:17:24 2017 +0300 ---------------------------------------------------------------------- ambari-web/app/messages.js | 2 ++ .../admin/stack_upgrade/stack_upgrade_wizard.hbs | 6 +++++- .../main/admin/stack_upgrade/upgrade_wizard_view.js | 16 ++++++++++++++++ 3 files changed, 23 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ambari/blob/2a57959f/ambari-web/app/messages.js ---------------------------------------------------------------------- diff --git a/ambari-web/app/messages.js b/ambari-web/app/messages.js index 2f07163..ba80702 100644 --- a/ambari-web/app/messages.js +++ b/ambari-web/app/messages.js @@ -1831,6 +1831,8 @@ Em.I18n.translations = { "Click on <b>Finalize</b> when you are ready to finalize the downgrade and commit to the new version." + " You are strongly encouraged to run tests on your cluster to ensure it is fully operational before finalizing." + " <b>You cannot go back to the original version once the downgrade is finalized.</b>", + 'admin.stackUpgrade.finalize.message.revertible': "The <b>{0}</b> upgrade to <b>{1}</b> is ready to be completed." + + " After finalization, the patch can be reverted from the Stacks and Versions page if it is no longer required.", 'admin.stackUpgrade.finalize.message.autoStart': "Please remember to re-enable Auto Start if you disabled it for upgrade.", 'admin.stackUpgrade.finalize.message.skippedServiceChecks': "During the upgrade, checks for the following services failed and were skipped:", 'admin.stackUpgrade.finalize.message.testServices': "You are strongly recommended to test these services before finalizing upgrade.", http://git-wip-us.apache.org/repos/asf/ambari/blob/2a57959f/ambari-web/app/templates/main/admin/stack_upgrade/stack_upgrade_wizard.hbs ---------------------------------------------------------------------- diff --git a/ambari-web/app/templates/main/admin/stack_upgrade/stack_upgrade_wizard.hbs b/ambari-web/app/templates/main/admin/stack_upgrade/stack_upgrade_wizard.hbs index a5525be..6d06f91 100644 --- a/ambari-web/app/templates/main/admin/stack_upgrade/stack_upgrade_wizard.hbs +++ b/ambari-web/app/templates/main/admin/stack_upgrade/stack_upgrade_wizard.hbs @@ -129,7 +129,11 @@ {{#if isDowngrade}} <p class="manual-steps-content">{{t admin.stackUpgrade.finalize.message.downgrade}}</p> {{else}} - <p class="manual-steps-content">{{t admin.stackUpgrade.finalize.message.upgrade}}</p> + {{#if view.isRevertibleUpgrade}} + <p class="manual-steps-content">{{{view.revertibleFinalizeMessage}}}</p> + {{else}} + <p class="manual-steps-content">{{t admin.stackUpgrade.finalize.message.upgrade}}</p> + {{/if}} {{/if}} <p class="manual-steps-content"><strong>{{t admin.stackUpgrade.finalize.message.autoStart}}</strong></p> http://git-wip-us.apache.org/repos/asf/ambari/blob/2a57959f/ambari-web/app/views/main/admin/stack_upgrade/upgrade_wizard_view.js ---------------------------------------------------------------------- diff --git a/ambari-web/app/views/main/admin/stack_upgrade/upgrade_wizard_view.js b/ambari-web/app/views/main/admin/stack_upgrade/upgrade_wizard_view.js index 3b269a0..10e34ab 100644 --- a/ambari-web/app/views/main/admin/stack_upgrade/upgrade_wizard_view.js +++ b/ambari-web/app/views/main/admin/stack_upgrade/upgrade_wizard_view.js @@ -208,6 +208,22 @@ App.upgradeWizardView = Em.View.extend({ isFinalizeItem: Em.computed.equalProperties('manualItem.context', 'controller.finalizeContext'), /** + * Upgrade of PATCH version is revertible + */ + isRevertibleUpgrade: function() { + var associatedVersion = this.get('controller.upgradeData.Upgrade.associated_version'); + var upgradeVersion = App.RepositoryVersion.find().findProperty('repositoryVersion', associatedVersion); + return ['PATCH'].contains(upgradeVersion.get('type')); + }.property('controller.upgradeData.Upgrade.associated_version'), + + revertibleFinalizeMessage: function() { + var associatedVersion = this.get('controller.upgradeData.Upgrade.associated_version'); + var upgradeVersion = App.RepositoryVersion.find().findProperty('repositoryVersion', associatedVersion); + return Em.I18n.t('admin.stackUpgrade.finalize.message.revertible') + .format(upgradeVersion.get('type'), upgradeVersion.get('displayName')); + }.property('controller.upgradeData.Upgrade.associated_version'), + + /** * label of Upgrade status * @type {string} */
