> On July 19, 2015, 1:28 a.m., Timothy Chen wrote: > > src/docker/executor.cpp, line 164 > > <https://reviews.apache.org/r/36585/diff/2/?file=1015868#file1015868line164> > > > > So what's special about the label that we also need to include the > > docker inspect data in there? Why can't the framework just use the data > > field? > > Kapil Arya wrote: > We want to expose the data via state.json so that Mesos-DNS can consume > it. However, the Master never stores TaskStatus::data and so it is not > exposed via state.json. > > Having said that, it looks like this RR would be a no-go. We would > probably end up using a Mesos hook module to add a Label with just the > container-IP which would then be exposed via state.json.
I see, yes we can't store this for all masters. - Timothy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36585/#review92196 ----------------------------------------------------------- On July 19, 2015, 12:38 a.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36585/ > ----------------------------------------------------------- > > (Updated July 19, 2015, 12:38 a.m.) > > > Review request for mesos, Benjamin Hindman, Ben Mahler, and Timothy Chen. > > > Bugs: MESOS-3061 > https://issues.apache.org/jira/browse/MESOS-3061 > > > Repository: mesos > > > Description > ------- > > This would allow Mesos-DNS to lookup container information such as its > IP address. > > NOTE: Apparently, the data size is about 15KB per `docker inspect` blob and > that might addup for large clusters. One suggestion is to expose only the IP > field(s) from: > > "NetworkSettings": { > "Bridge": "", > "EndpointID": > "c9d8bfb120503eda3dc88ffdcc5778cff4e7f05bb2e53f9b2fba82972ecfe3da", > "Gateway": "172.17.42.1", > "GlobalIPv6Address": "", > "GlobalIPv6PrefixLen": 0, > "HairpinMode": false, > "IPAddress": "172.17.0.5", > "IPPrefixLen": 16, > "IPv6Gateway": "", > "LinkLocalIPv6Address": "", > "LinkLocalIPv6PrefixLen": 0, > "MacAddress": "02:42:ac:11:00:05", > "NetworkID": > "f36472f7725507d90ab10215d35d020bf3eaf5ebdc024cb5b3ffff773967d9fc", > "PortMapping": null, > "Ports": {}, > "SandboxKey": "/var/run/docker/netns/3c3d4d0f5e18", > "SecondaryIPAddresses": null, > "SecondaryIPv6Addresses": null > }, > > An alternate is to use a hook module to decorate TaskStatus with IP > information extracted from the output of `docker inspect`. > > > Diffs > ----- > > src/docker/executor.cpp cdcd8ee7ad0b53748819bc1e565f708e30e99a5d > src/tests/docker_containerizer_tests.cpp > 6c6f4b7f30cec9b5bba77234b714e96289c82b43 > > Diff: https://reviews.apache.org/r/36585/diff/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Kapil Arya > >
