You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by at...@apache.org on 2018/12/28 09:24:55 UTC

[ambari] branch trunk updated: AMBARI-25034 Ambari is not updating core-site : "ha.zookeeper.quorum"once a zookeeper server is added or removed (asnaik) (#2715)

This is an automated email from the ASF dual-hosted git repository.

atkach pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ambari.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 5c73593  AMBARI-25034 Ambari is not updating core-site : "ha.zookeeper.quorum"once a zookeeper server is added or removed (asnaik) (#2715)
5c73593 is described below

commit 5c735930ede8298f0aa81f78641bf8b2f3efac36
Author: Akhil S Naik <as...@hortonworks.com>
AuthorDate: Fri Dec 28 14:54:51 2018 +0530

    AMBARI-25034 Ambari is not updating core-site : "ha.zookeeper.quorum"once a zookeeper server is added or removed (asnaik) (#2715)
    
    * AMBARI-25034 Ambari is not updating core-site : "ha.zookeeper.quorum" once a zookeeper server is added or removed (asnaik)
    
    *  AMBARI-25034 Ambari is not updating core-site : "ha.zookeeper.quorum" once a zookeeper server is added or removed - review comments (asnaik)
    
    * AMBARI-25034 Ambari is not updating core-site : "ha.zookeeper.quorum"once a zookeeper server is added or removed (asnaik)
---
 ambari-web/app/controllers/main/host/details.js | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/ambari-web/app/controllers/main/host/details.js b/ambari-web/app/controllers/main/host/details.js
index 31f64b7..46d9229 100644
--- a/ambari-web/app/controllers/main/host/details.js
+++ b/ambari-web/app/controllers/main/host/details.js
@@ -1730,10 +1730,15 @@ App.MainHostDetailsController = Em.Controller.extend(App.SupportClientConfigsDow
   constructZookeeperConfigUrlParams: function (data) {
     var urlParams = [];
     var services = App.Service.find();
+    var zooKeeperRelatedServices = this.get('zooKeeperRelatedServices').slice(0);
     if (App.get('isHaEnabled')) {
-      urlParams.push('(type=core-site&tag=' + data.Clusters.desired_configs['core-site'].tag + ')');
+      zooKeeperRelatedServices.push({
+        serviceName: 'HDFS',
+        typesToLoad: ['core-site'],
+        typesToSave: ['core-site']
+      });
     }
-    this.get('zooKeeperRelatedServices').forEach(function (service) {
+    zooKeeperRelatedServices.forEach(function (service) {
       if (services.someProperty('serviceName', service.serviceName)) {
         service.typesToLoad.forEach(function (type) {
           if (data.Clusters.desired_configs[type]) {
@@ -1762,7 +1767,15 @@ App.MainHostDetailsController = Em.Controller.extend(App.SupportClientConfigsDow
     this.updateZkConfigs(configs);
     var groups = [];
     var installedServiceNames = App.Service.find().mapProperty('serviceName');
-    this.get('zooKeeperRelatedServices').forEach(function (service) {
+    var zooKeeperRelatedServices = this.get('zooKeeperRelatedServices').slice(0);
+    if (App.get('isHaEnabled')) {
+      zooKeeperRelatedServices.push({
+        serviceName: 'HDFS',
+        typesToLoad: ['core-site'],
+        typesToSave: ['core-site']
+      });
+    }
+    zooKeeperRelatedServices.forEach(function (service) {
       if (installedServiceNames.contains(service.serviceName)) {
         var group = {
           properties: {},