mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Qian Zhang <zhq527...@gmail.com>
Subject Re: Review Request 60500: Introduced `--default_container_dns` agent flag.
Date Wed, 05 Jul 2017 02:48:21 GMT


> On July 4, 2017, 1:22 a.m., Avinash sridharan wrote:
> > docs/configuration.md
> > Lines 1329 (patched)
> > <https://reviews.apache.org/r/60500/diff/1/?file=1766230#file1766230line1329>
> >
> >     This is interesting. I think this example highlights a subtle difference between
the behavior of the DNS option for CNI and CNM. For CNI networks we don't really have a concept
of HOST or BRIDGE networks. However, for CNM networks since we support this option for HOST,
BRIDGE and USER networks, we should be explicit about this information. Also, we need to emphasize
that the `--dns` options for `HOST` and `BRIDGE` networks are supported only for docker 1.13
and above (for `USER` it has been available since 1.10). We probably want to enforce these
version checks for docker.
> 
> Qian Zhang wrote:
>     > However, for CNM networks since we support this option for HOST, BRIDGE and
USER networks, we should be explicit about this information. Also, we need to emphasize that
the --dns options for HOST and BRIDGE networks are supported only for docker 1.13 and above.
>     
>     I have a very old version of Docker (1.5.0) and I have checked its `docker run` command
also has the `--dns` option and it can work with `BRIDGE` network but not with `HOST` network.
Do you think we need to put such info into this help message (I mean emphasizing the minimal
Docker version for `--dns` support with `HOST`, `BRIDGE` and `USER` networks respectively)?
That may make this message too big.

By checking https://github.com/moby/moby/blob/master/CHANGELOG.md, I got the following info:
1. `--dns-opt` was introduced to `docker run` in Docker 1.9.0, see https://github.com/moby/moby/pull/16031.
2. `--dns` can be used with `HOST` network in Docker 1.12.0, see https://github.com/moby/moby/pull/22408.
3. `--dns-option` was introduced to `docker run` in Docker 1.13.0 and `--dns-opt` was hidden
(but it still can be used), see https://github.com/moby/moby/pull/28186.

I will do Docker version check based on the above info in https://reviews.apache.org/r/60558/.


- Qian


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60500/#review179520
-----------------------------------------------------------


On June 28, 2017, 10:57 p.m., Qian Zhang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60500/
> -----------------------------------------------------------
> 
> (Updated June 28, 2017, 10:57 p.m.)
> 
> 
> Review request for mesos, Avinash sridharan and Jie Yu.
> 
> 
> Bugs: MESOS-7709
>     https://issues.apache.org/jira/browse/MESOS-7709
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Introduced `--default_container_dns` agent flag.
> 
> 
> Diffs
> -----
> 
>   docs/configuration.md 0eb696a949003ff11831aed5e4f4ab384cf9992e 
>   src/messages/flags.hpp 70ad58c33067a0b058deafdcc4ffc5f554b6ca72 
>   src/messages/flags.proto e87075f19714739b93d9f4aa33ea8686a1bb5613 
>   src/slave/flags.hpp e75c1b4227b443aedf445921b3f2108d930c112c 
>   src/slave/flags.cpp c84aa6724170bba46b4444be8410b71d42a1626e 
> 
> 
> Diff: https://reviews.apache.org/r/60500/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Qian Zhang
> 
>


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