----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36529/#review91830 -----------------------------------------------------------
src/tests/cgroups_tests.cpp (line 1199) <https://reviews.apache.org/r/36529/#comment145471> I would add a comment that we're moving to the root cgroup. I'll add it for you this time. - Timothy Chen On July 15, 2015, 10:25 p.m., Jojy Varghese wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36529/ > ----------------------------------------------------------- > > (Updated July 15, 2015, 10:25 p.m.) > > > Review request for mesos and Timothy Chen. > > > Repository: mesos > > > Description > ------- > > cgroups::destroy does not succeed when there is a running task in the > cgroup.The test was failing as current task was in the cgroup being destroyed. > > > Diffs > ----- > > src/tests/cgroups_tests.cpp a371ee205f89de10e15902057848c35f7c278d77 > > Diff: https://reviews.apache.org/r/36529/diff/ > > > Testing > ------- > > > Thanks, > > Jojy Varghese > >
