> On Dec. 11, 2015, 11:19 p.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py, > > line 190 > > <https://reviews.apache.org/r/41284/diff/1/?file=1160530#file1160530line190> > > > > We should open a bug for the next release to allow recursive chown in > > the resource_management library itself. > > Andrew Onischuk wrote: > Sumit do you mean for 2.2.1 or 2.3?
I think the next minor release, so 2.3 will be OK. - Sumit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41284/#review110036 ----------------------------------------------------------- On Dec. 11, 2015, 11:17 p.m., Aravindan Vijayan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41284/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2015, 11:17 p.m.) > > > Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. > > > Bugs: AMBARI-14350 > https://issues.apache.org/jira/browse/AMBARI-14350 > > > Repository: ambari > > > Description > ------- > > PROBLEM > Steps: > Deployed cluster via blueprints. > Reset cluster (reset ambari-server, host cleanup). > Deployed cluster via UI. > Result: Ambari Metrics Service go down. > > BUG > AMS scripts do not change the ownership of certain directories to AMS user if > they exist already. > > FIX > Adding an explicit chown to take ownership of the required directories > (collector conf, collector checkpoint, HBase tmp, HBase conf, HBase root) > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py > 733fcc0 > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/hbase.py > 92d2bc4 > > ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_collector.py > e884367 > > Diff: https://reviews.apache.org/r/41284/diff/ > > > Testing > ------- > > Manual Testing done. Ambari server and agent unit tests pass. > > > Thanks, > > Aravindan Vijayan > >
