----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36816/#review110144 -----------------------------------------------------------
Patch looks great! Reviews applied: [36816] Passed command: export OS=ubuntu:14.04;export CONFIGURATION="--verbose";export COMPILER=gcc; ./support/docker_build.sh - Mesos ReviewBot On Dec. 12, 2015, 4:08 p.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36816/ > ----------------------------------------------------------- > > (Updated Dec. 12, 2015, 4:08 p.m.) > > > Review request for mesos, Adam B, Michael Park, and Timothy Chen. > > > Bugs: MESOS-2533 > https://issues.apache.org/jira/browse/MESOS-2533 > > > Repository: mesos > > > Description > ------- > > Support HTTP checks in Mesos health check program > > > Diffs > ----- > > include/mesos/mesos.proto 8ca213062c480f0266ffc51a621eb4a118140c77 > src/docker/executor.cpp 4042cec0acbe03d937ea3c53ffde745cbba552d2 > src/health-check/main.cpp 0beaed575ec865d81e6e3d83d8a0c894613acba4 > src/tests/health_check_tests.cpp 0fbccc373204d3b9431c614bdd6d046cc07e4566 > > Diff: https://reviews.apache.org/r/36816/diff/ > > > Testing > ------- > > * Add unit test cases: HealthCheckTest.HealthyTaskThroughHttp and > HealthCheckTest.HealthyTaskNotMatchHttpStatuses > make check > > > Thanks, > > haosdent huang > >