Skip to content
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

Merged

Conversation

AkihiroSuda
Copy link
Member

Close #40484

Note that the support for cgroup v2 isn't ready for production yet, regardless to rootful or rootless.

@AkihiroSuda AkihiroSuda added this to the 20.03.0 milestone Mar 31, 2020
@AkihiroSuda AkihiroSuda requested a review from tianon as a code owner March 31, 2020 07:49
@AkihiroSuda AkihiroSuda force-pushed the move-rootless-out-of-experimental branch from af30fe7 to 5e02308 Compare March 31, 2020 07:51
@AkihiroSuda
Copy link
Member Author

@thaJeztah PTAL?

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah
Copy link
Member

oh, actually; not a blocker, but can you also update the Jenkinsfile?

DOCKER_EXPERIMENTAL = '1'

@AkihiroSuda
Copy link
Member Author

Intentionally left Jenkinsfile as-is, because I thought we should still test other experimental features with rootless. WDYT?

@thaJeztah
Copy link
Member

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>
@AkihiroSuda
Copy link
Member Author

updated Jenkinsfile

@AkihiroSuda AkihiroSuda force-pushed the move-rootless-out-of-experimental branch from 5e02308 to d1304a8 Compare April 6, 2020 15:59
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

still LGTM

@AkihiroSuda
Copy link
Member Author

@tianon @tonistiigi @tiborvass PTAL

@thaJeztah
Copy link
Member

bringing this one in; @AkihiroSuda can you check if the documentation needs updating and open a "draft" PR for that if needed?

@thaJeztah thaJeztah merged commit a6a47d1 into moby:master Apr 7, 2020
AkihiroSuda added a commit to AkihiroSuda/docker that referenced this pull request Apr 29, 2020
Follow up to moby#40759

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rootless: graduate from experimental
3 participants