mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Avinash sridharan <avin...@mesosphere.io>
Subject Re: Review Request 51607: Used mesos-tcp-connect binary in TCP health checks.
Date Tue, 06 Sep 2016 18:54:24 GMT

-----------------------------------------------------------
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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message