----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/75174/ -----------------------------------------------------------
Review request for mesos and Benjamin Mahler. Repository: mesos Description ------- In cgroups1, we returned an error when we see that the container is sharing cgroups, as it would have no cgroups for update() to take. In cgroups2 we will mimick this behavior for containers that do not wish to have their own resource constraints, as we do not expect to call update() on those containers. Diffs ----- src/slave/containerizer/mesos/isolators/cgroups2/cgroups2.cpp d1507ecd979405d1fd4d56c5fcd990b22eb311f6 Diff: https://reviews.apache.org/r/75174/diff/1/ Testing ------- Compiles okay. When later patches are applied NamespacesIsolatorTest pass and NestedMesosContainerizerTests pass (except for the ones that check for subsystem hierarchies) Thanks, Jason Zhou
