----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/75145/ -----------------------------------------------------------
(Updated Aug. 6, 2024, 8:11 p.m.) Review request for mesos and Benjamin Mahler. Changes ------- Switch from returning failure to just return Nothing() if there is no file content to work with Repository: mesos Description ------- We currently do not have any method of recovering the device access states when the cgroups 2 isolator is atempting to recover containers. We add a recovery state here that makes use of the protobuf checkpoint files to ensure that the previous device accesses of cgroups can be restored. It will be used by the cgroups 2 isolator. Diffs (updated) ----- src/slave/containerizer/device_manager/device_manager.hpp 853350f70e12b992ef311a35c509a5dce8f2301a src/slave/containerizer/device_manager/device_manager.cpp e613323dc47a7980984426d37b6fc5cfc52dffe0 src/tests/device_manager_tests.cpp c4e9b8c58282b8d57b5ce88fefddd34c8ea30c77 Diff: https://reviews.apache.org/r/75145/diff/2/ Changes: https://reviews.apache.org/r/75145/diff/1-2/ Testing ------- Test added to check the functionality of recover() with container state. Tests pass, and the state was recovered. Thanks, Jason Zhou
