> On May 27, 2015, 3:11 p.m., Jonathan Hurley wrote: > > ambari-agent/src/main/python/ambari_agent/alerts/script_alert.py, lines > > 102-103 > > <https://reviews.apache.org/r/34712/diff/1/?file=972686#file972686line102> > > > > Why is this logger only set if there's a match for the scripts basedir; > > shouldn't the initialization of the ScriptAlert class do this in general?
this is resource_manamgement logger. Resource management is invalid without Environment. It does not make any sense to alter that logger when we don't have Environment - Andrew ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34712/#review85376 ----------------------------------------------------------- On May 27, 2015, 3:09 p.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34712/ > ----------------------------------------------------------- > > (Updated May 27, 2015, 3:09 p.m.) > > > Review request for Ambari and Vitalyi Brodetskyi. > > > Bugs: AMBARI-11349 > https://issues.apache.org/jira/browse/AMBARI-11349 > > > Repository: ambari > > > Description > ------- > > See attached files. This came from a devdeploy test where some service check > was failing due to lack of disk space (BUG - 37173). ambari-agent.out was not > the reason but was the largest contributor. > > Can you check if we can avoid logging to the out file unless there was an > error due to which this is expected? > > If out file does not roll this could be an issue in the long run. > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/alerts/script_alert.py 969a01f > > Diff: https://reviews.apache.org/r/34712/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
