----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51971/#review149259 -----------------------------------------------------------
Ship it! Ship It! - Sid Wagle On Sept. 16, 2016, 8:16 p.m., Aravindan Vijayan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51971/ > ----------------------------------------------------------- > > (Updated Sept. 16, 2016, 8:16 p.m.) > > > Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. > > > Bugs: AMBARI-17456 > https://issues.apache.org/jira/browse/AMBARI-17456 > > > Repository: ambari > > > Description > ------- > > Follow up bug fix for AMBARI-17456 (https://reviews.apache.org/r/51657/) > > BUG > The config_reader was expecting a string like "host1,host2" for > metrics_server_hosts, but params.py was passing in "['host1','host2']". Hence > emitter got bad hostnames like "['host1'" and "'host2']" after split. > > Also, brought back the "hostname" config in metric_monitor.ini since that can > be used to pass in the correct hostname as managed by ambari. That is used to > denote the current host rather than the collector host. > > > Diffs > ----- > > > ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/config_reader.py > 2e8a170 > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/templates/metric_monitor.ini.j2 > b011fd5 > > Diff: https://reviews.apache.org/r/51971/diff/ > > > Testing > ------- > > Manually tested > Unit tests pass. > > > Thanks, > > Aravindan Vijayan > >