> On July 28, 2017, 8:35 a.m., Avinash sridharan wrote: > > src/tests/containerizer/docker_containerizer_tests.cpp > > Lines 4357 (patched) > > <https://reviews.apache.org/r/60761/diff/3/?file=1782044#file1782044line4357> > > > > Can we have a test for checking validation error on setting up the HOST > > MODE?
Did you mean adding a test to test the validation code in the lambda that we added in `src/slave/flags.cpp` for the `--default_container_dns` flag? If so, then I think `docker_containerizer_tests.cpp` may not be a good place for such test because `docker_containerizer_tests.cpp` should contain the end-to-end tests, but for testing the validation code we do not even need to start the agent. Maybe just add a test in `flags_tests.cpp`? And in another hand, for the other agent flags which has the validation code in lambda, it seems there is no related test for them, so do we really need such test for testing the validation code of `--default_container_dns`? BTW, I was thinking to add a test for wildcard match of Docker user-defined network, but the problem is we can not assume any user-defined networks in the test env, but there must be a bridge network, so currently I only have the test for the bridge network. - Qian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60761/#review181628 ----------------------------------------------------------- On July 25, 2017, 2:07 p.m., Qian Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60761/ > ----------------------------------------------------------- > > (Updated July 25, 2017, 2:07 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 > ------- > > Added a test `DockerContainerizerTest.ROOT_DOCKER_DefaultDNS`. > > > Diffs > ----- > > src/tests/containerizer/docker_containerizer_tests.cpp > 1e85a79f812399270575ea4a64db10e72f40e648 > > > Diff: https://reviews.apache.org/r/60761/diff/3/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Qian Zhang > >
