----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21040/#review42084 -----------------------------------------------------------
Ship it! src/slave/containerizer/external_containerizer.cpp <https://reviews.apache.org/r/21040/#comment75833> We'll still probably want/need to specify something like the slave's ID and/or meta and/or work directory to enable the ECP to distinguish different slaves from one another. Probably best to put this in the environment. src/slave/containerizer/external_containerizer.cpp <https://reviews.apache.org/r/21040/#comment75834> s/foreach(/foreach (/ - Benjamin Hindman On May 6, 2014, 10 p.m., Till Toenshoff wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21040/ > ----------------------------------------------------------- > > (Updated May 6, 2014, 10 p.m.) > > > Review request for mesos and Niklas Nielsen. > > > Repository: mesos-git > > > Description > ------- > > Updates 'containers' implementation within the ExternalContainerizer to > forward the call to the ECP. > Adds new 'containers' proto message to containers.proto. > > Some minor cleanups like removing of needless namespace mentions that slipped > through. > > NOTE: The test_containerizer.py implementation of 'containers' is incomplete, > it merely poses as a mock-up. > NOTE: This does *not* fix MESOS-1257. > > > Diffs > ----- > > include/mesos/containerizer/containerizer.proto c926a7f > src/examples/python/test_containerizer.py 7bebfee > src/slave/containerizer/external_containerizer.hpp 8fdf097 > src/slave/containerizer/external_containerizer.cpp e51ac66 > > Diff: https://reviews.apache.org/r/21040/diff/ > > > Testing > ------- > > make (tests are currently disabled) > > > Thanks, > > Till Toenshoff > >
