Skip to content

rootless: graduate from experimental#40759

Merged
thaJeztah merged 2 commits intomoby:masterfrom
AkihiroSuda:move-rootless-out-of-experimental
Apr 7, 2020
Merged

rootless: graduate from experimental#40759
thaJeztah merged 2 commits intomoby:masterfrom
AkihiroSuda:move-rootless-out-of-experimental

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 <[email protected]>
DOCKER_EXPERIMENTAL is already set in the head of the file.

Signed-off-by: Akihiro Suda <[email protected]>
@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?

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