----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36846/#review93108 -----------------------------------------------------------
Ship it! Ship It! - Dmitro Lisnichenko On July 27, 2015, 9:33 a.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36846/ > ----------------------------------------------------------- > > (Updated July 27, 2015, 9:33 a.m.) > > > Review request for Ambari, Dmitro Lisnichenko and Vitalyi Brodetskyi. > > > Bugs: AMBARI-12548 > https://issues.apache.org/jira/browse/AMBARI-12548 > > > Repository: ambari > > > Description > ------- > > The agent executes the "df" command when it starts up. > In rare scenarios, this command can hang (can even hanging for several days > without any timeout). When this happens the agent won't move forward to the > process of registration, etc. > There's an ambari-server property called agent.check.mounts.timeout. This can > be defined to force a timeout. > However, we should always have a default timeout on the agent script side, > even if this property is not defined to avoid a situation where the agent is > completely hung. > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/Hardware.py 5567286 > ambari-agent/src/test/python/ambari_agent/TestHardware.py d1bc2e6 > ambari-server/src/test/python/custom_actions/TestCheckHost.py 024aab2 > > Diff: https://reviews.apache.org/r/36846/diff/ > > > Testing > ------- > > [INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0 > approved: 138 licence. > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 19.557s > [INFO] Finished at: Mon Jul 27 12:24:48 EEST 2015 > [INFO] Final Memory: 12M/192M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Dmytro Sen > >
