----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23462/#review47917 -----------------------------------------------------------
Ship it! src/slave/containerizer/composing.hpp <https://reviews.apache.org/r/23462/#comment84163> others are *_containerizer, e.g., mesos_containerizer, external_containerizer. What about composing_containerizer for consistency? src/slave/containerizer/composing.cpp <https://reviews.apache.org/r/23462/#comment84158> What happens if --containerizers is changed? i.e., you have running containers that were started with a containerizer that now isn't present? You're only recovering containers for containerizers that you're told to IIUC...? src/slave/containerizer/containerizer.cpp <https://reviews.apache.org/r/23462/#comment84161> order is incorrect, com < con src/slave/flags.hpp <https://reviews.apache.org/r/23462/#comment84154> Do you want to add something here about the order in which they are tried? - Ian Downes On July 14, 2014, 3:30 p.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23462/ > ----------------------------------------------------------- > > (Updated July 14, 2014, 3:30 p.m.) > > > Review request for mesos, Ian Downes and Jie Yu. > > > Bugs: MESOS-1527 > https://issues.apache.org/jira/browse/MESOS-1527 > > > Repository: mesos-git > > > Description > ------- > > Next part of MESOS-1527, see dependent review 23461 for the first phase. > > > Diffs > ----- > > src/Makefile.am 2c7bfc57103c47523466c297f2596eaf4eb55c3c > src/slave/containerizer/composing.hpp PRE-CREATION > src/slave/containerizer/composing.cpp PRE-CREATION > src/slave/containerizer/containerizer.cpp > 1b71f33b430645300d171eadc9f7de96d28adf60 > src/slave/flags.hpp 1fe7b7da5aaa221ed6b94203b1189a80b14edd25 > > Diff: https://reviews.apache.org/r/23462/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Hindman > >
