----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65728/#review198011 -----------------------------------------------------------
src/tests/health_check_tests.cpp Lines 2716 (patched) <https://reviews.apache.org/r/65728/#comment278176> Wouldn't this be `// TODO(akagup)` now? - Andrew Schwartzmeyer On Feb. 21, 2018, 6:17 a.m., Akash Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65728/ > ----------------------------------------------------------- > > (Updated Feb. 21, 2018, 6:17 a.m.) > > > Review request for mesos, Alexander Rukletsov, Andrew Schwartzmeyer, Gaston > Kleiman, and Joseph Wu. > > > Bugs: MESOS-4812 and MESOS-8498 > https://issues.apache.org/jira/browse/MESOS-4812 > https://issues.apache.org/jira/browse/MESOS-8498 > > > Repository: mesos > > > Description > ------- > > Added `ROOT_DOCKER_DockerHealthyTaskWithQuotedCommand` to make sure that > the docker executor properly deals with quotes in the command health > check to ensure that MESOS-4812 is fixed. > > > Diffs > ----- > > src/tests/health_check_tests.cpp 1893c85169f5e94e164434b93e6a24268224225d > > > Diff: https://reviews.apache.org/r/65728/diff/1/ > > > Testing > ------- > > > Thanks, > > Akash Gupta > >
