----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60500/#review180764 -----------------------------------------------------------
src/messages/flags.proto Lines 56 (patched) <https://reviews.apache.org/r/60500/#comment255988> Had an internal discussion on this with Jie, and seems like having different DNS options for CNI and CNM is not something we need to worry about, so we should merge the fields into one. - Avinash sridharan On July 5, 2017, 7:03 a.m., Qian Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60500/ > ----------------------------------------------------------- > > (Updated July 5, 2017, 7:03 a.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 007d0f556375f331148c9ad09be8b301ab31a01c > src/messages/flags.hpp 70ad58c33067a0b058deafdcc4ffc5f554b6ca72 > src/messages/flags.proto e87075f19714739b93d9f4aa33ea8686a1bb5613 > src/slave/flags.hpp c6803eb54e09a5497755e1e5fef2872193eacba6 > src/slave/flags.cpp 398768656b5fa3b7c85474de2b4b008bf7b85cb3 > > > Diff: https://reviews.apache.org/r/60500/diff/2/ > > > Testing > ------- > > > Thanks, > > Qian Zhang > >
