----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34484/#review84559 -----------------------------------------------------------
Master (998993d) is green with this patch. ./build-support/jenkins/build.sh However, it appears that it might lack test coverage. I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On May 20, 2015, 6:10 p.m., Brian Wickman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34484/ > ----------------------------------------------------------- > > (Updated May 20, 2015, 6:10 p.m.) > > > Review request for Aurora, Joe Smith and Vinod Kone. > > > Bugs: AURORA-698 > https://issues.apache.org/jira/browse/AURORA-698 > > > Repository: aurora > > > Description > ------- > > Daemonize all deadline calls in aurora executor. If we do not daemonize, > it's possible for the aurora executor to send TASK_KILLED and then block > indefinitely on shutdown. This way the aurora executor process will at least > exit, allow the cgroup to tear down all active processes. > > > Diffs > ----- > > src/main/python/apache/aurora/executor/aurora_executor.py > df0df0cba3269a137a370e2b4096fd61fd3af258 > > Diff: https://reviews.apache.org/r/34484/diff/ > > > Testing > ------- > > ./pants test src/test/python/apache/aurora/executor:: > > > Thanks, > > Brian Wickman > >