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

idtools: fix ToHost conversion #1057

Merged
merged 1 commit into from Nov 9, 2021

Conversation

giuseppe
Copy link
Member

@giuseppe giuseppe commented Nov 9, 2021

drop a special case for RootPair that prevented the conversion of the
UID/GID when it was matching the root user in the container.

Closes: #1056

Signed-off-by: Giuseppe Scrivano gscrivan@redhat.com

drop a special case for RootPair that prevented the conversion of the
UID/GID when it was matching the root user in the container.

Closes: containers#1056

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

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

LGTM

@rhatdan
Copy link
Member

rhatdan commented Nov 9, 2021

LGTM

@rhatdan rhatdan merged commit 38a952c into containers:main Nov 9, 2021
nalind added a commit to nalind/storage that referenced this pull request Mar 30, 2022
…sn't

The godoc for pkg/idtools.ToHost() mentions that it makes an exception
when either of the IDs to be mapped is 0, but that exception was removed
in containers#1057.

Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
nalind added a commit to nalind/storage that referenced this pull request Mar 30, 2022
…sn't

The godoc for pkg/idtools.ToHost() mentions that it makes an exception
when either of the IDs to be mapped is 0, but that special case was
removed in containers#1057.

Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants