----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46290/#review129401 -----------------------------------------------------------
There was a discussion in IRC about just configuring your health check to be `sudo ...`. I may have missed the resolution of that discussion, was that not a feasible approach? - Joshua Cohen On April 18, 2016, 6:33 p.m., Dmitriy Shirchenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46290/ > ----------------------------------------------------------- > > (Updated April 18, 2016, 6:33 p.m.) > > > Review request for Aurora, Bill Farner and Zameer Manji. > > > Bugs: AURORA-1666 > https://issues.apache.org/jira/browse/AURORA-1666 > > > Repository: aurora > > > Description > ------- > > adding --setuid-health-checks flag to not demote health check user:group to > role > > > Diffs > ----- > > RELEASE-NOTES.md a0536ec352119952d4d58aa5f36e5e9a7b7d2e6e > src/main/python/apache/aurora/executor/bin/thermos_executor_main.py > 40a0cd6396a78c591debf5e2be11363ecf496231 > src/main/python/apache/aurora/executor/common/health_checker.py > d8675beb8f16ebdd2d6946367784411fe84a5cfc > src/test/python/apache/aurora/executor/common/test_health_checker.py > 4ab7a2fab651abd5ab8a0f312d10c444800e8b7a > > Diff: https://reviews.apache.org/r/46290/diff/ > > > Testing > ------- > > - end to end tests > - added unit tests > > > Thanks, > > Dmitriy Shirchenko > >