> On May 9, 2014, 6:33 p.m., Benjamin Hindman wrote: > > src/slave/containerizer/external_containerizer.cpp, line 714 > > <https://reviews.apache.org/r/21040/diff/2/?file=573989#file573989line714> > > > > 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.
Yes, this has actually bitten me in the upcoming test_containerizer C++ implementation when getting exercised via SlaveRecoverTests - great catch, thanks Ben! - Till ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21040/#review42084 ----------------------------------------------------------- On May 11, 2014, 8:24 p.m., Till Toenshoff wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21040/ > ----------------------------------------------------------- > > (Updated May 11, 2014, 8:24 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. > A complete test_containerizer C++ implementation is upcoming to fix these > notes. > > > 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 > >
