Skip to content

client:improve mount error message#11861

Merged
AkihiroSuda merged 1 commit intocontainerd:mainfrom
ningmingxiao:mount_err_detail
May 22, 2025
Merged

client:improve mount error message#11861
AkihiroSuda merged 1 commit intocontainerd:mainfrom
ningmingxiao:mount_err_detail

Conversation

@ningmingxiao
Copy link
Copy Markdown
Contributor

@ningmingxiao ningmingxiao commented May 15, 2025

if use containerd api in container to create container will return error:

failed to create container: failed to mount /tmp/containerd-mount530130800: no such file or directory.

It is misleading because /tmp/containerd-mount530130800 exists.

ref:#11849
after this commit

failed to create container: failed to mount /tmp/containerd-mount4215502267: mount source: overlay, target: /tmp/containerd-mount4215502267, fstype: overlay, flags: 0, data: lowerdir=/var/lib/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots/31/fs:/var/lib/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots/1/fs,index=off, err: no such file or directory

@k8s-ci-robot
Copy link
Copy Markdown

Hi @ningmingxiao. Thanks for your PR.

I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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-sigs/prow repository.

@ningmingxiao
Copy link
Copy Markdown
Contributor Author

@AkihiroSuda @mxpv @mikebrow

@ningmingxiao ningmingxiao force-pushed the mount_err_detail branch 3 times, most recently from e71f2d1 to c44cf3e Compare May 15, 2025 10:59
Comment thread core/mount/mount_linux.go Outdated
@ningmingxiao
Copy link
Copy Markdown
Contributor Author

can this pr be merged? @samuelkarp @dmcgowan @AkihiroSuda it's easy to review.

@github-project-automation github-project-automation Bot moved this from Needs Triage to Review In Progress in Pull Request Review May 22, 2025
@AkihiroSuda AkihiroSuda added this pull request to the merge queue May 22, 2025
@AkihiroSuda
Copy link
Copy Markdown
Member

/cherry-pick release/2.1

@k8s-infra-cherrypick-robot
Copy link
Copy Markdown

@AkihiroSuda: once the present PR merges, I will cherry-pick it on top of release/2.1 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release/2.1

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-sigs/prow repository.

Merged via the queue into containerd:main with commit b337c00 May 22, 2025
97 of 100 checks passed
@github-project-automation github-project-automation Bot moved this from Review In Progress to Done in Pull Request Review May 22, 2025
@k8s-infra-cherrypick-robot
Copy link
Copy Markdown

@AkihiroSuda: new pull request created: #11884

Details

In response to this:

/cherry-pick release/2.1

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-sigs/prow repository.

k8s-infra-cherrypick-robot pushed a commit to k8s-infra-cherrypick-robot/containerd that referenced this pull request Jun 4, 2025
Commit fb6dd2c ("client:improve mount error message") changed
the return value into a wraped error message, which causes the
ENOTBLK check doesn't work.

Reported-by: Aadhar Agarwal <[email protected]>
Fixes: containerd#11861
Signed-off-by: Gao Xiang <[email protected]>
aadhar-agarwal pushed a commit to aadhar-agarwal/containerd that referenced this pull request Aug 7, 2025
Commit fb6dd2c ("client:improve mount error message") changed
the return value into a wraped error message, which causes the
ENOTBLK check doesn't work.

Reported-by: Aadhar Agarwal <[email protected]>
Fixes: containerd#11861
Signed-off-by: Gao Xiang <[email protected]>
aadhar-agarwal pushed a commit to aadhar-agarwal/containerd that referenced this pull request Aug 7, 2025
Commit fb6dd2c ("client:improve mount error message") changed
the return value into a wraped error message, which causes the
ENOTBLK check doesn't work.

Reported-by: Aadhar Agarwal <[email protected]>
Fixes: containerd#11861
Signed-off-by: Gao Xiang <[email protected]>
Cindia-blue pushed a commit to Cindia-blue/containerd that referenced this pull request Aug 31, 2025
Commit fb6dd2c ("client:improve mount error message") changed
the return value into a wraped error message, which causes the
ENOTBLK check doesn't work.

Reported-by: Aadhar Agarwal <[email protected]>
Fixes: containerd#11861
Signed-off-by: Gao Xiang <[email protected]>
Cindia-blue pushed a commit to Cindia-blue/containerd that referenced this pull request Sep 1, 2025
Commit fb6dd2c ("client:improve mount error message") changed
the return value into a wraped error message, which causes the
ENOTBLK check doesn't work.

Reported-by: Aadhar Agarwal <[email protected]>
Fixes: containerd#11861
Signed-off-by: Gao Xiang <[email protected]>
Cindia-blue pushed a commit to Cindia-blue/containerd that referenced this pull request Oct 20, 2025
Commit fb6dd2c ("client:improve mount error message") changed
the return value into a wraped error message, which causes the
ENOTBLK check doesn't work.

Reported-by: Aadhar Agarwal <[email protected]>
Fixes: containerd#11861
Signed-off-by: Gao Xiang <[email protected]>
Cindia-blue pushed a commit to Cindia-blue/containerd that referenced this pull request Nov 8, 2025
Commit fb6dd2c ("client:improve mount error message") changed
the return value into a wraped error message, which causes the
ENOTBLK check doesn't work.

Reported-by: Aadhar Agarwal <[email protected]>
Fixes: containerd#11861
Signed-off-by: Gao Xiang <[email protected]>
@chrishenzie chrishenzie added the cherry-picked/2.1.x PR commits are cherry picked into the release/2.1 branch label Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-picked/2.1.x PR commits are cherry picked into the release/2.1 branch needs-ok-to-test size/S

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

7 participants