From 060df0121a05f0c05d45916603dbb0286182a979 Mon Sep 17 00:00:00 2001 From: Ignatious Johnson Christopher Date: Fri, 16 Sep 2016 11:53:10 -0700 Subject: [PATCH] Issue: cassandra ip is set as zookeeper ip's and it was working earlier as the cassanda and zookeeper was running in same node, now after moving zookeeper to config node, need to correct it and point to config node. Fix: Setting zookeeper ip's recived from commandline is used to set the zookeeper parameter in contrail-snmp-collector.conf and contrail-topology.conf files. Change-Id: If58b89cebfb4941b4e481972f83f657f3a3d0c54 Closes-Bug: 1623791 --- contrail_provisioning/collector/setup.py | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/contrail_provisioning/collector/setup.py b/contrail_provisioning/collector/setup.py index 79d4de0c..0e0201b2 100755 --- a/contrail_provisioning/collector/setup.py +++ b/contrail_provisioning/collector/setup.py @@ -50,7 +50,7 @@ def __init__(self, args_str = None): cassandra_port = '9160' self.cassandra_server_list = [(cassandra_server_ip, cassandra_port) for cassandra_server_ip in self._args.cassandra_ip_list] zookeeper_port = '2181' - self.zookeeper_server_list = None + self.zookeeper_server_list = [] if self._args.zookeeper_ip_list: self.zookeeper_server_list = [(zookeeper_server_ip, zookeeper_port) for \ zookeeper_server_ip in self._args.zookeeper_ip_list] @@ -189,8 +189,8 @@ def fixup_contrail_alarm_gen(self): kafka_broker_list_str) #prepare zklist - zk_list = [server[0] + ":2181" for server in self.cassandra_server_list] - zk_list_str = ' '.join(map(str, zk_list)) + zk_list_str = ' '.join('%s:%s' % zookeeper_server + for zookeeper_server in self.zookeeper_server_list) self.set_config(ALARM_GEN_CONF_FILE, 'DEFAULTS', 'zk_list', zk_list_str) @@ -207,7 +207,8 @@ def fixup_contrail_snmp_collector(self): local("echo 'mibs +ALL' > /etc/snmp/snmp.conf") local("[ -f %s ] || > %s" % (conf_fl, conf_fl)) self.set_config(conf_fl, 'DEFAULTS', 'zookeeper', - ','.join('%s:2181' % cs[0] for cs in self.cassandra_server_list)) + ','.join('%s:%s' % zookeeper_server + for zookeeper_server in self.zookeeper_server_list)) self.set_config(conf_fl, 'DISCOVERY', 'disc_server_ip', self._args.cfgm_ip) self.set_config(conf_fl, 'DISCOVERY', 'disc_server_port', '5998') @@ -232,7 +233,8 @@ def fixup_contrail_topology(self): with settings(warn_only=True): local("[ -f %s ] || > %s" % (conf_fl, conf_fl)) self.set_config(conf_fl, 'DEFAULTS', 'zookeeper', - ','.join('%s:2181' % cs[0] for cs in self.cassandra_server_list)) + ','.join('%s:%s' % zookeeper_server + for zookeeper_server in self.zookeeper_server_list)) self.set_config(conf_fl, 'DISCOVERY', 'disc_server_ip', self._args.cfgm_ip) self.set_config(conf_fl, 'DISCOVERY', 'disc_server_port', '5998')