----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52057/#review149755 -----------------------------------------------------------
Updated summary and description to say "nested" instead of "child" containers. include/mesos/agent/agent.proto (lines 58 - 60) <https://reviews.apache.org/r/52057/#comment217451> We are doing verbs before the nouns in all our calls. So these should be LAUNCH_NESTED_CONTAINER WAIT_NESTED_CONTAINER KILL_NESTED_CONTAINER include/mesos/agent/agent.proto (lines 102 - 122) <https://reviews.apache.org/r/52057/#comment217452> these message names should change once you change the enums. include/mesos/v1/agent/agent.proto (line 103) <https://reviews.apache.org/r/52057/#comment217456> The first part of this sentence should be moved to "ContainerID" proto? include/mesos/v1/agent/agent.proto (line 161) <https://reviews.apache.org/r/52057/#comment217458> ditto. WAIT_NESTED_CONTAINER include/mesos/v1/agent/agent.proto (line 271) <https://reviews.apache.org/r/52057/#comment217459> ditto. include/mesos/v1/agent/agent.proto (line 289) <https://reviews.apache.org/r/52057/#comment217460> ditto. src/slave/http.cpp (lines 402 - 409) <https://reviews.apache.org/r/52057/#comment217461> ditto. rename. - Vinod Kone On Sept. 20, 2016, 7:34 p.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52057/ > ----------------------------------------------------------- > > (Updated Sept. 20, 2016, 7:34 p.m.) > > > Review request for mesos, Anand Mazumdar, Benjamin Hindman, Jie Yu, and Vinod > Kone. > > > Bugs: MESOS-2449 > https://issues.apache.org/jira/browse/MESOS-2449 > > > Repository: mesos > > > Description > ------- > > This allows the executor to manage child containers within its > container. This is useful for the implementation of task groups > ("pod"-like containers) as well as for executors that would like > to implement isolation between sub-tasks. > > This initial patch provides the ability to launch, wait, and > destroy child containers. This was added as an agent API rather > than an executor API in order to enable operators to inject > subcontainers for future use-cases. For example, an operator > may want to inject a debugging container at run-time. > > > Diffs > ----- > > include/mesos/agent/agent.proto 5b91677487f8e23301e67af14c2115c6b4a533ac > include/mesos/v1/agent/agent.proto 8145669073553dc8aa56cfe2c0a0b756d70fee0e > src/slave/http.cpp 2cbf3c57e294f0d47d64915371b4a6bbf9cdfc0a > src/slave/slave.hpp e659c44fcc415f10a6980e7b29cc1ce536dfe657 > src/slave/validation.cpp a9f318214182a87783a985cd8495a0ec00c95378 > > Diff: https://reviews.apache.org/r/52057/diff/ > > > Testing > ------- > > Tests will be added alongside the implementation. > > > Thanks, > > Benjamin Mahler > >