----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51966/#review149270 -----------------------------------------------------------
src/slave/containerizer/mesos/isolators/gpu/isolator.cpp (lines 194 - 198) <https://reviews.apache.org/r/51966/#comment216837> Why removing the cgroup existence check? I think this still applies, especially we have to deal with the upgrade case? - Jie Yu On Sept. 16, 2016, 7:08 p.m., Kevin Klues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51966/ > ----------------------------------------------------------- > > (Updated Sept. 16, 2016, 7:08 p.m.) > > > Review request for mesos, Benjamin Mahler and Jie Yu. > > > Bugs: MESOS-6188 > https://issues.apache.org/jira/browse/MESOS-6188 > > > Repository: mesos > > > Description > ------- > > Updated the 'gpu/nvidia' isolator to be nested container aware. > > > Diffs > ----- > > src/slave/containerizer/mesos/isolators/gpu/isolator.cpp > 1383ba2dbb2be99b3eca0f2f62386438827e0bd6 > > Diff: https://reviews.apache.org/r/51966/diff/ > > > Testing > ------- > > > Thanks, > > Kevin Klues > >
