Skip to content

storage: fix UID/GID map handling for shifting layers#734

Open
giuseppe wants to merge 2 commits intocontainers:mainfrom
giuseppe:fix-map-handling
Open

storage: fix UID/GID map handling for shifting layers#734
giuseppe wants to merge 2 commits intocontainers:mainfrom
giuseppe:fix-map-handling

Conversation

@giuseppe
Copy link
Copy Markdown
Member

@giuseppe giuseppe commented Apr 1, 2026

when the driver supports shifting, set the rechown target to identity instead of the container's maps.

Closes: https://redhat.atlassian.net/browse/RHEL-160859
Assisted-by: Claude Opus 4.6
Signed-off-by: Giuseppe Scrivano gscrivan@redhat.com

giuseppe added 2 commits April 1, 2026 15:34
Remove the canUseShifting branch from CreateContainer and always store
the caller's IDMappingOptions.

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
when the driver supports shifting, set the rechown target to identity
instead of the container's maps.

Closes: https://redhat.atlassian.net/browse/RHEL-160859
Assisted-by: Claude Opus 4.6
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@mtrmac
Copy link
Copy Markdown
Contributor

mtrmac commented Apr 2, 2026

Note to self: Also #735.

@TomSweeneyRedHat
Copy link
Copy Markdown
Member

LGTM
but definitely want some other head nods

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.

3 participants