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

migrate to github.com/moby/sys/user/userns #5189

Merged
merged 2 commits into from
Jul 25, 2024

Conversation

thaJeztah
Copy link
Member

vendor: github.com/moby/sys/user v0.2.0

full diff: moby/sys@user/v0.1.0...user/v0.2.0

migrate to github.com/moby/sys/user/userns

The userns package in libcontainer was integrated into the moby/sys/user
module at commit 3778ae603c706494fd1e2c2faf83b406e38d687d.

The userns package is used in many places, and currently either depends
on runc/libcontainer, or on containerd, both of which have a complex
dependency tree. This patch is part of a series of patches to unify the
implementations, and to migrate toward that implementation to simplify
the dependency tree.

full diff: moby/sys@user/v0.1.0...user/v0.2.0

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah self-assigned this Jul 25, 2024
@crazy-max crazy-max added the area/dependencies Pull requests that update a dependency file label Jul 25, 2024
@thaJeztah
Copy link
Member Author

Oh! I should update depguard as well; let me do so

The userns package in libcontainer was integrated into the moby/sys/user
module at commit [3778ae603c706494fd1e2c2faf83b406e38d687d][1].

The userns package is used in many places, and currently either depends
on runc/libcontainer, or on containerd, both of which have a complex
dependency tree. This patch is part of a series of patches to unify the
implementations, and to migrate toward that implementation to simplify
the dependency tree.

[1]: opencontainers/runc@3778ae6

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Member Author

done 👍

@crazy-max crazy-max merged commit db3ab54 into moby:master Jul 25, 2024
75 checks passed
@thaJeztah thaJeztah deleted the migrate_userns branch August 6, 2024 10:52
@thaJeztah
Copy link
Member Author

⚠️ after further discussion, we decided to move this package to a separate module instead of as part of the moby/sys/user module; we should get that change in before tagging a new release;

cc @tonistiigi @crazy-max FYI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies Pull requests that update a dependency file
Projects
Development

Successfully merging this pull request may close these issues.

2 participants