> On Dec. 9, 2015, 1:26 a.m., Sid Wagle wrote: > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py, > > line 911 > > <https://reviews.apache.org/r/41109/diff/1/?file=1156612#file1156612line911> > > > > Validation looks good, but do we add this property correctly if AMS is > > present and Storm service exists?
Yes. It is part of recommendStormConfigurations. - Aravindan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41109/#review109438 ----------------------------------------------------------- On Dec. 9, 2015, 12:02 a.m., Aravindan Vijayan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41109/ > ----------------------------------------------------------- > > (Updated Dec. 9, 2015, 12:02 a.m.) > > > Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. > > > Bugs: AMBARI-14278 > https://issues.apache.org/jira/browse/AMBARI-14278 > > > Repository: ambari > > > Description > ------- > > The Storm Metrics reporter property (metrics.reporter.register) is a known > config item that needs to be added manually when upgraded from Ambari > versions before 2.0.0 to any version with AMS. If that is not done, there is > a stack advisor exception that is caused. > > FIX > Added a check for the presence of the property. If the property is present, > we add it to validation items. Else, we dont add to validation item > > Steps to work around the issue > 1. Add the following property to storm-site.xml from Ambari Web > metrics.reporter.register = > org.apache.hadoop.metrics2.sink.storm.StormTimelineMetricsReporter > 2. Restart Storm service. > > > Diffs > ----- > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py > f3e6f8b > > Diff: https://reviews.apache.org/r/41109/diff/ > > > Testing > ------- > > ambari-server python unit tests pass. > > Manually tested. > > > Thanks, > > Aravindan Vijayan > >
