-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Support LinuxDeviceCgroup
updates
#3401
Comments
FYI: |
@utam0k hm, that is the containerd API basically allowing to update using a Resource object correct? I guess it still requires the runc changes to make the device part actually work. |
@agners As far as I know, |
@utam0k from what I understand |
@agners That's right. But |
To be able to adjust device permissions at runtime, this issue seeks to implement support for Linux Device CGroup updates. It is a prerequisite to get such support in Moby/Docker, see also moby/moby#42255.
Currently supplying
devices
torunc update --resources -
does not adjust cgroup(v2) device permissions for an existing container. Some issues with device updates have been outlined in #2366 and addressed in various PRs linked to that issue.To implement support updates would need to pass
devices
to the internalconfig.Cgroups.Resources.Devices
structure. Besides update also setsconfig.Cgroups.SkipDevices
to false explicitly, which probably would need changing.I guess the semantics of update would need to be defined exactly. A simplistic approach would be to implement a behavior similar to what
CreateCgroupConfig
implements: Clear the current list of devices, append the list provided by the update command, then append the defaultDevs.In PR #2951 @cyphar added a comment saying:
What are those design issues exactly?
The text was updated successfully, but these errors were encountered: