You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by xi...@apache.org on 2015/10/27 21:58:05 UTC

[2/2] ambari git commit: AMBARI-13571. Upgrade: downgrade API invocation needs to specify the type.(xiwang)

AMBARI-13571. Upgrade: downgrade API invocation needs to specify the type.(xiwang)


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/d3eb8a8e
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/d3eb8a8e
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/d3eb8a8e

Branch: refs/heads/trunk
Commit: d3eb8a8ee621221edc10ffae685f664c9b97be08
Parents: 92cde3d
Author: Xi Wang <xi...@apache.org>
Authored: Mon Oct 26 16:19:48 2015 -0700
Committer: Xi Wang <xi...@apache.org>
Committed: Tue Oct 27 11:56:17 2015 -0700

----------------------------------------------------------------------
 .../main/admin/stack_and_upgrade_controller.js            |  3 ++-
 ambari-web/app/utils/ajax/ajax.js                         |  3 ++-
 .../main/admin/stack_and_upgrade_controller_test.js       | 10 ++++++----
 3 files changed, 10 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/d3eb8a8e/ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js b/ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js
index 21f6164..3b54778 100644
--- a/ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js
+++ b/ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js
@@ -462,7 +462,8 @@ App.MainAdminStackAndUpgradeController = Em.Controller.extend(App.LocalStorage,
         from: App.RepositoryVersion.find().findProperty('displayName', this.get('upgradeVersion')).get('repositoryVersion'),
         value: currentVersion.repository_version,
         label: currentVersion.repository_name,
-        isDowngrade: true
+        isDowngrade: true,
+        upgradeType: this.get('upgradeType')
       },
       success: 'upgradeSuccessCallback',
       callback: function() {

http://git-wip-us.apache.org/repos/asf/ambari/blob/d3eb8a8e/ambari-web/app/utils/ajax/ajax.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/utils/ajax/ajax.js b/ambari-web/app/utils/ajax/ajax.js
index f845381..d1fa528 100644
--- a/ambari-web/app/utils/ajax/ajax.js
+++ b/ambari-web/app/utils/ajax/ajax.js
@@ -1633,7 +1633,8 @@ var urls = {
           },
           "Upgrade": {
             "from_version": data.from,
-            "repository_version": data.value
+            "repository_version": data.value,
+            "upgrade_type": data.upgradeType
           }
         })
       }

http://git-wip-us.apache.org/repos/asf/ambari/blob/d3eb8a8e/ambari-web/test/controllers/main/admin/stack_and_upgrade_controller_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/controllers/main/admin/stack_and_upgrade_controller_test.js b/ambari-web/test/controllers/main/admin/stack_and_upgrade_controller_test.js
index 56bf0f7..2c15790 100644
--- a/ambari-web/test/controllers/main/admin/stack_and_upgrade_controller_test.js
+++ b/ambari-web/test/controllers/main/admin/stack_and_upgrade_controller_test.js
@@ -533,7 +533,7 @@ describe('App.MainAdminStackAndUpgradeController', function() {
         }),
         Em.Object.create({
           displayName: Em.I18n.t('admin.stackVersions.version.upgrade.upgradeOptions.EU.title'),
-          type: 'NON-ROLLING'
+          type: 'NON_ROLLING'
         })
       ];
       controller.upgradeSuccessCallback(data, {}, {label: 'HDP-2.2.1', isDowngrade: true});
@@ -788,7 +788,7 @@ describe('App.MainAdminStackAndUpgradeController', function() {
         }),
         Em.Object.create({
           displayName: Em.I18n.t('admin.stackVersions.version.upgrade.upgradeOptions.EU.title'),
-          type: 'NON-ROLLING',
+          type: 'NON_ROLLING',
           icon: "icon-bolt",
           description: Em.I18n.t('admin.stackVersions.version.upgrade.upgradeOptions.EU.description'),
           selected: false,
@@ -843,16 +843,18 @@ describe('App.MainAdminStackAndUpgradeController', function() {
     });
     it("make ajax call", function() {
       controller.set('upgradeVersion', 'HDP-2.3');
+      controller.set('upgradeType', 'NON_ROLLING');
       controller.downgrade(Em.Object.create({
         repository_version: '2.2',
         repository_name: 'HDP-2.2'
       }), {context: 'context'});
       expect(controller.abortUpgrade.calledOnce).to.be.true;
       expect(App.ajax.send.getCall(0).args[0].data).to.eql({
+        from: '2.3',
         value: '2.2',
         label: 'HDP-2.2',
-        from: '2.3',
-        isDowngrade: true
+        isDowngrade: true,
+        upgradeType: "NON_ROLLING"
       });
       expect(App.ajax.send.getCall(0).args[0].name).to.eql('admin.downgrade.start');
       expect(App.ajax.send.getCall(0).args[0].sender).to.eql(controller);