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 <akihiro.suda.cz@hco.ntt.co.jp>
(cherry picked from commit d5d5ccc)
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>

@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

4 participants