----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55505/#review162682 -----------------------------------------------------------
Fix it, then Ship it! ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py (line 35) <https://reviews.apache.org/r/55505/#comment234026> Even in unit tests, don't hardcode /usr/hdp/ - Alejandro Fernandez On Jan. 13, 2017, 4:46 p.m., Vishal Suvagia wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55505/ > ----------------------------------------------------------- > > (Updated Jan. 13, 2017, 4:46 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Gautam Borad, Madhan > Neethiraj, Mugdha Varadkar, Oliver Szabo, and Sumit Mohanty. > > > Bugs: AMBARI-19524 > https://issues.apache.org/jira/browse/AMBARI-19524 > > > Repository: ambari > > > Description > ------- > > Hardcoding Atlas conf_dir path does lead to Upgrade Downgrade scenarios, need > to update Atlac Conf path as per stack structures > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/status_params.py > 1fd1b9b > ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py > f2fec70 > ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py 584763d > > Diff: https://reviews.apache.org/r/55505/diff/ > > > Testing > ------- > > Verified fresh installation on centos-6. > > > Thanks, > > Vishal Suvagia > >