AMBARI-14738. zookeeper.connect property not updated in Kafka after adding one 
more ZK server (alexantonenko)


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

Branch: refs/heads/branch-dev-patch-upgrade
Commit: a7a979623179f060fabba493b70e211d0af1356f
Parents: 5990fab
Author: Alex Antonenko <hiv...@gmail.com>
Authored: Wed Jan 20 17:16:17 2016 +0200
Committer: Alex Antonenko <hiv...@gmail.com>
Committed: Wed Jan 20 18:10:48 2016 +0200

----------------------------------------------------------------------
 ambari-web/app/controllers/main/host/details.js      | 15 +++++++++++++++
 ambari-web/app/messages.js                           |  4 ++--
 .../configs/add_component_config_initializer.js      |  3 ++-
 3 files changed, 19 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/a7a97962/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 f9c1951..d325cde 100644
--- a/ambari-web/app/controllers/main/host/details.js
+++ b/ambari-web/app/controllers/main/host/details.js
@@ -1296,6 +1296,9 @@ App.MainHostDetailsController = 
Em.Controller.extend(App.SupportClientConfigsDow
     if (services.someProperty('serviceName', 'ACCUMULO')) {
       urlParams.push('(type=accumulo-site&tag=' + 
data.Clusters.desired_configs['accumulo-site'].tag + ')');
     }
+    if (services.someProperty('serviceName', 'KAFKA')) {
+      urlParams.push('(type=kafka-broker&tag=' + 
data.Clusters.desired_configs['kafka-broker'].tag + ')');
+    }
     return urlParams;
   },
 
@@ -1362,6 +1365,18 @@ App.MainHostDetailsController = 
Em.Controller.extend(App.SupportClientConfigsDow
         }
       );
     }
+    if (installedServiceNames.contains('KAFKA')) {
+      groups.push(
+        {
+          properties: {
+            'kafka-broker': configs['kafka-broker']
+          },
+          properties_attributes: {
+            'kafka-broker': attributes['kafka-broker']
+          }
+        }
+      );
+    }
     this.saveConfigsBatch(groups, 'ZOOKEEPER_SERVER');
   },
 

http://git-wip-us.apache.org/repos/asf/ambari/blob/a7a97962/ambari-web/app/messages.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/messages.js b/ambari-web/app/messages.js
index 82797a7..fac2893 100644
--- a/ambari-web/app/messages.js
+++ b/ambari-web/app/messages.js
@@ -2328,14 +2328,14 @@ Em.I18n.translations = {
   'host.host.componentFilter.slave':'Slave Components',
   'host.host.componentFilter.client':'Client Components',
   'hosts.host.deleteComponent.popup.msg1':'Are you sure you want to delete 
{0}?',
-  'hosts.host.deleteComponent.popup.deleteZooKeeperServer':'Deleting 
<i>ZooKeeper Server</i> may reconfigure such 
properties:<ul><li>ha.zookeeper.quorum</li><li>hbase.zookeeper.quorum</li><li>templeton.zookeeper.hosts</li><li>yarn.resourcemanager.zk-address</li><li>hive.zookeeper.quorum</li><li>hive.cluster.delegation.token.store.zookeeper.connectString</li><li>storm.zookeeper.servers</li><li>instance.zookeeper.host</li></ul>',
+  'hosts.host.deleteComponent.popup.deleteZooKeeperServer':'Deleting 
<i>ZooKeeper Server</i> may reconfigure such 
properties:<ul><li>zookeeper.connect</li><li>ha.zookeeper.quorum</li><li>hbase.zookeeper.quorum</li><li>templeton.zookeeper.hosts</li><li>yarn.resourcemanager.zk-address</li><li>hive.zookeeper.quorum</li><li>hive.cluster.delegation.token.store.zookeeper.connectString</li><li>storm.zookeeper.servers</li><li>instance.zookeeper.host</li></ul>',
   'hosts.host.deleteComponent.popup.deleteRangerKMSServer': 'Deleting 
<i>Ranger KMS Server</i> may reconfigure such 
properties:<ul><li>hadoop.security.key.provider.path</li><li>dfs.encryption.key.provider.uri</li>',
   'hosts.host.deleteComponent.popup.warning':'<b>WARNING!</b> Delete the last 
<i>{0}</i> component in the cluster?</br>Deleting the last component in the 
cluster could result in permanent loss of service data.',
   'hosts.host.deleteComponent.popup.confirm':'Confirm Delete',
   'hosts.host.installComponent.popup.confirm':'Confirm Install',
   'hosts.host.installComponent.msg':'Are you sure you want to install {0}?',
   'hosts.host.addComponent.msg':'Are you sure you want to add {0}?',
-  'hosts.host.addComponent.ZOOKEEPER_SERVER':'Adding ZooKeeper Server may 
reconfigure such 
properties:<ul><li>ha.zookeeper.quorum</li><li>hbase.zookeeper.quorum</li><li>templeton.zookeeper.hosts</li><li>yarn.resourcemanager.zk-address</li><li>hive.zookeeper.quorum</li><li>hive.cluster.delegation.token.store.zookeeper.connectString</li><li>storm.zookeeper.servers</li><li>instance.zookeeper.host</li></ul>',
+  'hosts.host.addComponent.ZOOKEEPER_SERVER':'Adding ZooKeeper Server may 
reconfigure such 
properties:<ul><li>zookeeper.connect</li><li>ha.zookeeper.quorum</li><li>hbase.zookeeper.quorum</li><li>templeton.zookeeper.hosts</li><li>yarn.resourcemanager.zk-address</li><li>hive.zookeeper.quorum</li><li>hive.cluster.delegation.token.store.zookeeper.connectString</li><li>storm.zookeeper.servers</li><li>instance.zookeeper.host</li></ul>',
   'hosts.host.addComponent.NIMBUS': 'Adding Nimbus will reconfigure 
<b>nimbus.seeds</b>, <b>topology.min.replication.count</b>, 
<b>topology.max.replication.wait.time.sec</b> properties if they are defined.',
   'hosts.host.addComponent.RANGER_KMS_SERVER': 'Adding Ranger KMS Server may 
reconfigure such 
properties:<ul><li>hadoop.security.key.provider.path</li><li>dfs.encryption.key.provider.uri</li>',
   'hosts.host.addComponent.deleteHostWithZooKeeper':'Deleting host with 
ZooKeeper Server may reconfigure such 
properties:<ul><li>ha.zookeeper.quorum</li><li>hbase.zookeeper.quorum</li><li>templeton.zookeeper.hosts</li><li>yarn.resourcemanager.zk-address</li><li>hive.zookeeper.quorum</li><li>hive.cluster.delegation.token.store.zookeeper.connectString</li></ul>',

http://git-wip-us.apache.org/repos/asf/ambari/blob/a7a97962/ambari-web/app/utils/configs/add_component_config_initializer.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/utils/configs/add_component_config_initializer.js 
b/ambari-web/app/utils/configs/add_component_config_initializer.js
index 6fc505c..5bbe02c 100644
--- a/ambari-web/app/utils/configs/add_component_config_initializer.js
+++ b/ambari-web/app/utils/configs/add_component_config_initializer.js
@@ -47,6 +47,7 @@ App.AddComponentConfigInitializer = 
App.HaConfigInitializerClass.extend(App.Host
    */
   __defaultInitializers: function() {
     return {
+      'zookeeper.connect': this.getHDPStackOnlyHostsPortConfig('2.2', 
'ZOOKEEPER_SERVER', '', '', ',', 'zkClientPort', true),
       'ha.zookeeper.quorum': 
this.getNameNodeHAOnlyHostsPortConfig('ZOOKEEPER_SERVER', '', '', ',', 
'zkClientPort', true),
       'hbase.zookeeper.quorum': 
this.getHostsListComponentConfig('ZOOKEEPER_SERVER', true),
       'instance.zookeeper.host': 
this.getHostsWithPortConfig('ZOOKEEPER_SERVER', '', '', ',', 'zkClientPort', 
true),
@@ -56,7 +57,6 @@ App.AddComponentConfigInitializer = 
App.HaConfigInitializerClass.extend(App.Host
       'hive.zookeeper.quorum': this.getHDPStackOnlyHostsPortConfig('2.2', 
'ZOOKEEPER_SERVER', '', '', ',', 'zkClientPort', true),
       'hadoop.registry.zk.quorum': this.getHDPStackOnlyHostsPortConfig('2.2', 
'ZOOKEEPER_SERVER', '', '', ',', 'zkClientPort', true),
       'nimbus.seeds': 
this.getHostsListComponentJSONStringifiedConfig('NIMBUS', true),
-      'hadoop.proxyuser.{{hiveUser}}.hosts': 
this.getComponentsHostsConfig(['HIVE_SERVER', 'WEBHCAT_SERVER', 
'HIVE_METASTORE'], false),
       'hadoop.proxyuser.{{webhcatUser}}.hosts': 
this.getComponentsHostsConfig(['HIVE_SERVER', 'WEBHCAT_SERVER', 
'HIVE_METASTORE'], false, true),
       'hadoop.proxyuser.{{hiveUser}}.hosts': 
this.getComponentsHostsConfig(['HIVE_SERVER', 'WEBHCAT_SERVER', 
'HIVE_METASTORE'], false, true),
       'hive.metastore.uris': this.getHostsWithPortConfig(['WEBHCAT_SERVER', 
'HIVE_METASTORE'], 'thrift://', '', ',thrift://', 'hiveMetastorePort', true)
@@ -277,6 +277,7 @@ App.AddComponentConfigInitializer = 
App.HaConfigInitializerClass.extend(App.Host
  */
 App.AddZooKeeperComponentsInitializer = 
App.AddComponentConfigInitializer.create({
   initializeForProperties: [
+    'zookeeper.connect',
     'ha.zookeeper.quorum',
     'hbase.zookeeper.quorum',
     'instance.zookeeper.host',

Reply via email to