You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by rz...@apache.org on 2015/09/04 02:26:54 UTC
ambari git commit: AMBARI-13005. Usablity: Deleting a host should not
reconfigure config sites (rzang)
Repository: ambari
Updated Branches:
refs/heads/trunk a5c247f32 -> 5fa8db23c
AMBARI-13005. Usablity: Deleting a host should not reconfigure config sites (rzang)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/5fa8db23
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/5fa8db23
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/5fa8db23
Branch: refs/heads/trunk
Commit: 5fa8db23c706cc1bbb5dcc8d22dc7bde27cd60e1
Parents: a5c247f
Author: Richard Zang <rz...@apache.org>
Authored: Thu Sep 3 17:25:32 2015 -0700
Committer: Richard Zang <rz...@apache.org>
Committed: Thu Sep 3 17:25:32 2015 -0700
----------------------------------------------------------------------
ambari-web/app/controllers/main/host/details.js | 7 -------
ambari-web/test/controllers/main/host/details_test.js | 2 +-
2 files changed, 1 insertion(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ambari/blob/5fa8db23/ambari-web/app/controllers/main/host/details.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/main/host/details.js b/ambari-web/app/controllers/main/host/details.js
index 7b8e6a9..20d5a67 100644
--- a/ambari-web/app/controllers/main/host/details.js
+++ b/ambari-web/app/controllers/main/host/details.js
@@ -2198,13 +2198,6 @@ App.MainHostDetailsController = Em.Controller.extend({
deleteHostSuccessCallback: function (data, rq, requestBody) {
var self = this;
App.router.get('updateController').updateHost(function () {
- if (!!App.Service.find().findProperty('serviceName', 'HIVE')) {
- self.loadConfigs('loadHiveConfigs');
- }
- var callback = function () {
- self.loadConfigs();
- };
- self.isServiceMetricsLoaded(callback);
App.router.transitionTo('hosts.index');
});
if (!!(requestBody && requestBody.hostName))
http://git-wip-us.apache.org/repos/asf/ambari/blob/5fa8db23/ambari-web/test/controllers/main/host/details_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/controllers/main/host/details_test.js b/ambari-web/test/controllers/main/host/details_test.js
index bfd1d53..7ddb2dd 100644
--- a/ambari-web/test/controllers/main/host/details_test.js
+++ b/ambari-web/test/controllers/main/host/details_test.js
@@ -2572,7 +2572,7 @@ describe('App.MainHostDetailsController', function () {
controller.deleteHostSuccessCallback();
expect(App.router.get.calledWith('updateController')).to.be.true;
expect(mock.updateHost.calledOnce).to.be.true;
- expect(controller.loadConfigs.called).to.be.true;
+ expect(controller.loadConfigs.called).to.be.false;
expect(App.router.transitionTo.calledWith('hosts.index')).to.be.true;
expect(App.router.get.calledWith('clusterController')).to.be.true;
expect(mock.getAllHostNames.calledOnce).to.be.true;