----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26383/#review56438 -----------------------------------------------------------
Thanks for working with us on the requirements. One drive-by comment, i'll let those on the People line take the rest. src/main/python/apache/aurora/executor/common/health_checker.py <https://reviews.apache.org/r/26383/#comment96743> This seems worthy of log.info - Bill Farner On Oct. 13, 2014, 7:27 p.m., David Pan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26383/ > ----------------------------------------------------------- > > (Updated Oct. 13, 2014, 7:27 p.m.) > > > Review request for Aurora, Joe Smith, Brian Wickman, and Zameer Manji. > > > Bugs: AURORA-795 > https://issues.apache.org/jira/browse/AURORA-795 > > > Repository: aurora > > > Description > ------- > > The health check disabler allows health checks for a job to be snoozed > temporarily by touching a snooze file in the job's sandbox. The appropriate > unit tests were modified/added. > > The corresponding JIRA ticket is the following: > https://issues.apache.org/jira/browse/AURORA-795 > > > Diffs > ----- > > src/main/python/apache/aurora/executor/common/health_checker.py > 4980411c847d12655cbb363404707ebd9f0bd163 > src/test/python/apache/aurora/executor/common/BUILD > c7f7a003c865d479ba6e3cd7b5349322f884f653 > src/test/python/apache/aurora/executor/common/test_health_checker.py > aa36415fa891fc523a3a376ffeca5d3cd5ceabec > > Diff: https://reviews.apache.org/r/26383/diff/ > > > Testing > ------- > > On vagrant in ~/aurora, I ran > ./pants src/test/python/apache/aurora/executor:: > > > Thanks, > > David Pan > >