----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64104/#review192559 -----------------------------------------------------------
src/tests/health_check_tests.cpp Line 565 (original), 565 (patched) <https://reviews.apache.org/r/64104/#comment270798> The docker tests should be disabled for now since they don't work. For example, this tests tries running the `alpine` image on Windows. I'm working on enabling the docker health checks on Windows. src/tests/health_check_tests.cpp Line 2492 (original), 2492 (patched) <https://reviews.apache.org/r/64104/#comment270797> Should remove the `DOCKER_` since it uses the mesos containerizer. - Akash Gupta On Nov. 30, 2017, 5:18 p.m., John Kordich wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64104/ > ----------------------------------------------------------- > > (Updated Nov. 30, 2017, 5:18 p.m.) > > > Review request for mesos, Akash Gupta, Andrew Schwartzmeyer, Jeff Coffler, > and Joseph Wu. > > > Bugs: MESOS-6709 > https://issues.apache.org/jira/browse/MESOS-6709 > > > Repository: mesos > > > Description > ------- > > Added dependency of curl to agent. Enabled most health check tests. > > > Diffs > ----- > > src/slave/CMakeLists.txt 6f08f3dc95f3a1408cbef7b8f0b0cc2522558924 > src/tests/health_check_tests.cpp c0dcba265363f2149b217b189ee5a8e925e40ea1 > > > Diff: https://reviews.apache.org/r/64104/diff/1/ > > > Testing > ------- > > There are two health check tests which are not enabled yet, because they > require the IOSwitchboard to be ported first. We can't resolve MESOS-6709 > quite yet until that is addressed. The rest of the tests pass on both > Linux/Windows, as mentioned in #64102 > > > Thanks, > > John Kordich > >