----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51607/#review147887 -----------------------------------------------------------
src/health-check/health_checker.cpp (line 117) <https://reviews.apache.org/r/51607/#comment215179> Shouldn't we check is this directory exists before creating this `HealthCheckerProcess`? src/health-check/health_checker.cpp (line 520) <https://reviews.apache.org/r/51607/#comment215186> This more of a question: Why is the --host hard coded to the `DEFAULT_DOMAIN`. What if the container is binding to a specific IP address, instead of any ADDR_ANY? Also, in case of the docker containerizer, the `tcpConnect` binary is going to be running in the host network namespace so connecting to localhost might work for cases where the port-mapping is setup for the containers, but don't think it will work for cases where the container (docker container) has no port mapping but has it own network namespace. This will happen say with Calico or even the DC/OS overlay. - Avinash sridharan On Sept. 2, 2016, 4:42 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51607/ > ----------------------------------------------------------- > > (Updated Sept. 2, 2016, 4:42 p.m.) > > > Review request for mesos, Avinash sridharan, Benjamin Mahler, Gastón Kleiman, > and haosdent huang. > > > Bugs: MESOS-6119 > https://issues.apache.org/jira/browse/MESOS-6119 > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/docker/executor.cpp f5981b080c0b81ef7025527ab26b36d7a01f009b > src/health-check/health_checker.hpp > 392b4d5bd1e5831994b9366c1eb5a2911e19860f > src/health-check/health_checker.cpp > f373df19fc8af8e9650be61e3b101e89362a67cd > src/launcher/executor.cpp 5370634ef9e6f3ac9717fed71f6a77707026a16a > > Diff: https://reviews.apache.org/r/51607/diff/ > > > Testing > ------- > > make check on Mac OS 10.11.6 > > > Thanks, > > Alexander Rukletsov > >
