Skip to content

[20.10 backport] pkg/archive: allow mknodding FIFO inside userns#41957

Merged
tiborvass merged 1 commit into
moby:20.10from
AkihiroSuda:cherrypick-41892-2010
Feb 11, 2021
Merged

[20.10 backport] pkg/archive: allow mknodding FIFO inside userns#41957
tiborvass merged 1 commit into
moby:20.10from
AkihiroSuda:cherrypick-41892-2010

Conversation

@AkihiroSuda

@AkihiroSuda AkihiroSuda commented Feb 2, 2021

Copy link
Copy Markdown
Member

Cherry-pick #41892

- What I did
Fix #41803

Also attempt to mknod devices.
Mknodding devices are likely to fail, but still worth trying when running with a seccomp user notification.

- How I did it
By allowing mknodding FIFO inside userns.

- How to verify it
docker pull docker.io/gitlab/gitlab-ee:13.6.3-ee.0 with rootless

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

🐧

Fix moby#41803

Also attempt to mknod devices.
Mknodding devices are likely to fail, but still worth trying when
running with a seccomp user notification.

Signed-off-by: Akihiro Suda <[email protected]>
(cherry picked from commit d5d5ccc)
Signed-off-by: Akihiro Suda <[email protected]>

@cpuguy83 cpuguy83 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@tiborvass
tiborvass merged commit 11443bc into moby:20.10 Feb 11, 2021
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.

4 participants