DevinLeamy commented on code in PR #552:
URL: https://github.com/apache/mesos/pull/552#discussion_r1557888760
##########
src/slave/containerizer/mesos/linux_launcher.hpp:
##########
@@ -25,6 +25,17 @@ namespace mesos {
namespace internal {
namespace slave {
+// DO(dleamy): Move this into the header file.
+struct CgroupsInfo
+{
+ // Flag indicating whether cgroups v2 is used.
+ bool cgroupsV2;
+
+ Option<std::string> freezerHierarchy;
+
+ Option<std::string> systemdHierarchy;
+};
+
Review Comment:
Would like to move this into the source file and only have a forward
declaration in the header. Doing that was giving compilation errors. Perhaps
you have some insight here? @bmahler
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
