----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63515/ -----------------------------------------------------------
Review request for mesos, Gilbert Song and Jie Yu. Bugs: MESOS-7305 https://issues.apache.org/jira/browse/MESOS-7305 Repository: mesos Description ------- These three isolators refered to the ExecutorInfo field of the given ContainerConfig to determine the total resources to isolate. This changes these isolators to refer instead to the Resources field of the given ContainerConfig because the resource amounts should be identical (enforced by the caller to Containerizer::launch, such as the agent) and in order to support standalone containers, which specify resources but no ExecutorInfo. Diffs ----- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp 549a4551736d1b88a79f275a2d6e19c586c49f26 src/slave/containerizer/mesos/isolators/gpu/isolator.cpp 25636b50314cb4ea53d1931c97c87c65ecb658a8 src/slave/containerizer/mesos/isolators/xfs/disk.cpp 2113f8665e4e84bdf12868f605b480522816a9ba Diff: https://reviews.apache.org/r/63515/diff/1/ Testing ------- make check sudo src/mesos-tests On... OSX and Ubuntu 16 Thanks, Joseph Wu