Skip to content

Defer uid lookups on Darwin#8312

Merged
mxpv merged 1 commit intocontainerd:mainfrom
mxpv:oci
Mar 27, 2023
Merged

Defer uid lookups on Darwin#8312
mxpv merged 1 commit intocontainerd:mainfrom
mxpv:oci

Conversation

@mxpv
Copy link
Copy Markdown
Member

@mxpv mxpv commented Mar 27, 2023

Mounts are not supported on Darwin, so carry the trick used on Windows for Darwin (e.g. defer uid:gid lookups).

@mxpv mxpv requested review from crosbymichael and dcantah March 27, 2023 17:19
@mxpv mxpv added cherry-pick/1.7.x Change to be cherry picked to release/1.7 branch easy-to-review Easy to review labels Mar 27, 2023
Comment thread oci/spec_opts.go
Copy link
Copy Markdown
Member

@dcantah dcantah left a comment

Choose a reason for hiding this comment

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

LGTM after my one bit of feedback

Signed-off-by: Maksym Pavlenko <[email protected]>
@k8s-ci-robot
Copy link
Copy Markdown

@mxpv: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-containerd-node-e2e 87346df link true /test pull-containerd-node-e2e

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mxpv
Copy link
Copy Markdown
Member Author

mxpv commented Mar 27, 2023

/retest

@mxpv mxpv merged commit 4a00e06 into containerd:main Mar 27, 2023
@mxpv mxpv deleted the oci branch March 27, 2023 22:34
@mxpv mxpv removed the cherry-pick/1.7.x Change to be cherry picked to release/1.7 branch label Mar 27, 2023
@dmcgowan dmcgowan added the cherry-picked/1.7.x PR commits are cherry-picked into release/1.7 branch label May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-picked/1.7.x PR commits are cherry-picked into release/1.7 branch easy-to-review Easy to review

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants