-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[cgroupv2] Add workspace cgroup evacuation #8578
Conversation
Codecov Report
@@ Coverage Diff @@
## main #8578 +/- ##
===========================================
+ Coverage 12.31% 27.94% +15.62%
===========================================
Files 20 85 +65
Lines 1161 13874 +12713
===========================================
+ Hits 143 3877 +3734
- Misses 1014 9674 +8660
- Partials 4 323 +319
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
This PR introduces a cgroup v2 structure in workspaces which lends itself to more rootless workloads within the workspace. For example:
cgroup v2 - new structure - Watch Video
The structure it produces looks as follows:
Note that the container-cgroup does not change ownership, but only the child cgroups do. This way, users cannot escape the control imposed by the container runtime.
Related Issue(s)
Fixes #8567
How to test
/sys/fs/cgroup
Release Notes