> On Aug. 12, 2024, 3:52 p.m., Benjamin Mahler wrote: > > src/tests/containerizer/cgroups_isolator_tests.cpp > > Lines 2790 (patched) > > <https://reviews.apache.org/r/75163/diff/1/?file=2292711#file2292711line2790> > > > > do I understand correctly that all of the cgroups2::xxx functions > > support `cgroup` being relative or absolute?
Any function that calls cgroups2::read/write will go through the cgroups2::path function which will be able to handle both relative and absolute paths. See: https://reviews.apache.org/r/75081/ > On Aug. 12, 2024, 3:52 p.m., Benjamin Mahler wrote: > > src/tests/containerizer/cgroups_isolator_tests.cpp > > Line 2790 (original), 2792-2801 (patched) > > <https://reviews.apache.org/r/75163/diff/1/?file=2292711#file2292711line2792> > > > > this seems to be a test that checks that all the subsystems / > > controllers can be loaded? > > > > ``` > > // This test verifies all the local enabled cgroups subsystems > > // can be automatically loaded by the cgroup isolator. > > ``` > > > > so don't you want to check that all the various controllers we support > > are enabled? We can have a static list of controllers we can expect to have enabled, and fail the test if any of them are missing - Jason ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/75163/#review226817 ----------------------------------------------------------- On Aug. 12, 2024, 2:44 p.m., Jason Zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/75163/ > ----------------------------------------------------------- > > (Updated Aug. 12, 2024, 2:44 p.m.) > > > Review request for mesos and Benjamin Mahler. > > > Repository: mesos > > > Description > ------- > > currently, the ROOT_CGROUPS_AutoLoadSubsystems test is failing because > it is checking for hierarchies for subsystems enabled under > 'cgroups/all'. In cgroups2 we cannot perform this check because of the > unified hierarchy. > > Hence we skip this hierarchy check and instead check that all available > cgroups2 controllers are enabled by reading cgroup.controllers and > cgroup.subtree_control. > > > Diffs > ----- > > src/tests/containerizer/cgroups_isolator_tests.cpp > f2d6d274146ed1ef8f560fbba0da18051e0f1281 > > > Diff: https://reviews.apache.org/r/75163/diff/1/ > > > Testing > ------- > > CgroupsIsolatorTest.ROOT_CGROUPS_AutoLoadSubsystems passes > > > Thanks, > > Jason Zhou > >
