mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Anand Mazumdar <>
Subject Re: Review Request 52164: Added `ContainerID` field to the executor `SUBSCRIBED` event.
Date Fri, 23 Sep 2016 03:24:52 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Sept. 23, 2016, 3:24 a.m.)

Review request for mesos and Vinod Kone.


Review comments

Bugs: MESOS-6227

Repository: mesos


This change adds the field `ContainerID` to the executor `SUBSCRIBED`
event. This allows the default executor/other custom executors to
then launch nested containers. This is needed as the executors need
to let the agent know of the parent container ID when they make the
`LAUNCH_NESTED_CONTAINER` call. This change also implements the
logic on the agent to send the event to the executor.

Diffs (updated)

  include/mesos/executor/executor.proto 0d31406a70d6b7d9030c1f157a7869a2e8f1879f 
  include/mesos/v1/executor/executor.proto f661b71cecb96a9971c687f117625ef242749394 
  src/slave/slave.cpp 11e9c8af87aa5153f72f2a20cc578fe3d729b153 
  src/tests/executor_http_api_tests.cpp fd322ee267b2f0d4bf250db15027d61f1fc74a57 



make check


Anand Mazumdar

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message