ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Aravindan Vijayan" <avija...@hortonworks.com>
Subject Review Request 41109: AMBARI-14278 : On a cluster with both AMS and Storm, stack advisor throws error in validateStormConfigurations after upgrade if the metrics.reporter.register property is not present
Date Tue, 08 Dec 2015 23:14:17 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41109/
-----------------------------------------------------------

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


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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message