----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74871/#review226222 -----------------------------------------------------------
src/Makefile.am Lines 2863 (patched) <https://reviews.apache.org/r/74871/#comment314468> nit: nbd, but can you align the backspace here? (1 less tab character) src/tests/containerizer/cgroups2_tests.cpp Lines 17-22 (patched) <https://reviews.apache.org/r/74871/#comment314467> doesn't look like these includes are needed yet? let's add them only when we need them (as a general rule) src/tests/containerizer/cgroups2_tests.cpp Lines 32 (patched) <https://reviews.apache.org/r/74871/#comment314464> maybe your editor can automatically strip trailing whitespace? - Benjamin Mahler On Feb. 12, 2024, 5:02 p.m., Devin Leamy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74871/ > ----------------------------------------------------------- > > (Updated Feb. 12, 2024, 5:02 p.m.) > > > Review request for mesos. > > > Repository: mesos > > > Description > ------- > > Introduces cgroups2_tests.cpp which will hold the cgroups2 tests. > Adds a test that checks that cgroups2 is enabled on the testing host. > > > Diffs > ----- > > src/Makefile.am d1e16d1c97d41107328bcd8cd6eefb5dacdf557d > src/tests/CMakeLists.txt 6beb74e382acbda5bca816709312b00785673c4a > src/tests/containerizer/cgroups2_tests.cpp PRE-CREATION > > > Diff: https://reviews.apache.org/r/74871/diff/1/ > > > Testing > ------- > > > Thanks, > > Devin Leamy > >
