Skip to content

overlay: treat EACCES/EPERM metacopy probe failure as unsupported - #1068

Merged
giuseppe merged 1 commit into
podman-container-tools:mainfrom
giuseppe:overlay-check-rootless
Aug 12, 2026
Merged

overlay: treat EACCES/EPERM metacopy probe failure as unsupported#1068
giuseppe merged 1 commit into
podman-container-tools:mainfrom
giuseppe:overlay-check-rootless

Conversation

@giuseppe

Copy link
Copy Markdown
Contributor

doesMetacopy() already treats EINVAL from the probe mount as "metacopy not supported" and lets the driver initialise with metacopy off. In a restricted unprivileged user namespace the plain overlay mount can succeed while the metacopy probe mount is refused with EACCES (or EPERM), which was propagated out of Init() as a fatal error and left the driver unusable, even though it would work perfectly with metacopy disabled.

Closes: #1053

Comment thread storage/drivers/overlay/check.go Outdated
@giuseppe
giuseppe force-pushed the overlay-check-rootless branch from b1c0ebc to f2d1be5 Compare August 12, 2026 10:05
doesMetacopy() already treats EINVAL from the probe mount as "metacopy
not supported" and lets the driver initialise with metacopy off. In a
restricted unprivileged user namespace the plain overlay mount can
succeed while the metacopy probe mount is refused with EACCES (or EPERM),
which was propagated out of Init() as a fatal error and left the driver
unusable, even though it would work perfectly with metacopy disabled.

Closes: podman-container-tools#1053

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe
giuseppe force-pushed the overlay-check-rootless branch from f2d1be5 to c5a9f3f Compare August 12, 2026 10:10

@Luap99 Luap99 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

@nalind nalind left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@giuseppe
giuseppe merged commit 97c1ece into podman-container-tools:main Aug 12, 2026
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

storage Related to "storage" package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

overlay: a metacopy=on probe failure makes the driver unusable, even though metacopy is never supported for unprivileged mounts

3 participants