----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39152/#review102108 -----------------------------------------------------------
Ship it! LGTM src/tests/containerizer/mesos_containerizer_tests.cpp (line 366) <https://reviews.apache.org/r/39152/#comment159621> Not a huge fan of this hackery but this approach is also used by other similar tests. I wonder if we should create a separate test fixture for this test/such tests in general ? Also , wondering what do other projects do or what is the recommended way ? - Anand Mazumdar On Oct. 9, 2015, 8:34 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39152/ > ----------------------------------------------------------- > > (Updated Oct. 9, 2015, 8:34 p.m.) > > > Review request for mesos, Adam B and Kapil Arya. > > > Bugs: MESOS-3553 > https://issues.apache.org/jira/browse/MESOS-3553 > > > Repository: mesos > > > Description > ------- > > If DNS is not available on the agent node and a task is launched which > explicitly specifies the executor's environment, LIBPROCESS_IP will not be > passed through and the default hostname lookup after spawning the executor > process will throw an error. This patch alters the agent to always pass > LIBPROCESS_IP, even when the executor environment is specified. > > > Diffs > ----- > > src/slave/containerizer/containerizer.cpp > 25c87e9f948b7efe8b9a853c403bee69982d6c4c > src/tests/containerizer/mesos_containerizer_tests.cpp > 5bc7d408bda0c249e1b66747d8bd87e688362e6c > > Diff: https://reviews.apache.org/r/39152/diff/ > > > Testing > ------- > > `make check` > > > Thanks, > > Greg Mann > >