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