-
Notifications
You must be signed in to change notification settings - Fork 18.7k
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
rootless: graduate from experimental #40759
rootless: graduate from experimental #40759
Conversation
af30fe7
to
5e02308
Compare
@thaJeztah PTAL? |
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
oh, actually; not a blocker, but can you also update the Jenkinsfile? Line 411 in c239bbb
|
Intentionally left Jenkinsfile as-is, because I thought we should still test other experimental features with rootless. WDYT? |
Gotcha; no need to change; I don't think it's even needed, because the global env-var already enables it (so should be inherited); https://github.com/moby/moby/blob/master/Jenkinsfile#L24 |
Close moby#40484 Note that the support for cgroup v2 isn't ready for production yet, regardless to rootful or rootless. Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
DOCKER_EXPERIMENTAL is already set in the head of the file. Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
updated Jenkinsfile |
5e02308
to
d1304a8
Compare
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.
still LGTM
@tianon @tonistiigi @tiborvass PTAL |
bringing this one in; @AkihiroSuda can you check if the documentation needs updating and open a "draft" PR for that if needed? |
Follow up to moby#40759 Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Close #40484
Note that the support for cgroup v2 isn't ready for production yet, regardless to rootful or rootless.