----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49828/#review144282 -----------------------------------------------------------
src/slave/containerizer/mesos/isolators/cgroups/subsystem.cpp (lines 80 - 92) <https://reviews.apache.org/r/49828/#comment210271> Suppose we use `cgroups/cpu` as isolation first, then restart the agent with `cgroups/cpu,cgroups/mem` in `isolation` flag. For those recovered containers, because they don't appear in the memory subsystem before, they should return `Failure` here. - haosdent huang On July 31, 2016, 5:49 p.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49828/ > ----------------------------------------------------------- > > (Updated July 31, 2016, 5:49 p.m.) > > > Review request for mesos, Gilbert Song, Jie Yu, Qian Zhang, and Zhengju Sha. > > > Bugs: MESOS-5039 > https://issues.apache.org/jira/browse/MESOS-5039 > > > Repository: mesos > > > Description > ------- > > Added default methods implementations for `Subsystem` base class. > > > Diffs > ----- > > src/slave/containerizer/mesos/isolators/cgroups/subsystem.hpp > 5f52a076a1fa3a21d886cb961ddeed5046a38d7c > src/slave/containerizer/mesos/isolators/cgroups/subsystem.cpp > a30ecafcbecc9d3b6eeea2b04dcb4d278750af41 > > Diff: https://reviews.apache.org/r/49828/diff/ > > > Testing > ------- > > > Thanks, > > haosdent huang > >
