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

fix nerdctl ps show nothing when timeout #4010

Merged
merged 1 commit into from
Mar 19, 2025

Conversation

ningmingxiao
Copy link
Contributor

@ningmingxiao ningmingxiao commented Mar 17, 2025

[root@RISC-v-test-minion-0-1 nmx]# nerdctl  -n k8s.io ps
CONTAINER ID    IMAGE    COMMAND    CREATED    STATUS    PORTS    NAMES

containerd many log

Mar 17 15:40:42 RISC-v-test-minion-0-1 containerd[2922522]: time="2025-03-17T15:40:41.993695418+08:00" level=error msg="get state for 6366492fc9778f66e90c228faee9154484d8cd6c0440eb1770bd91c93a60eeb1" error="context canceled: unknown


@ningmingxiao
Copy link
Contributor Author

ningmingxiao commented Mar 17, 2025

we can delete default tomeout 5s.
because containerd config already have config to set getState timeout.

 "io.containerd.timeout.task.state" = "60s"

@AkihiroSuda

@ningmingxiao
Copy link
Contributor Author

ningmingxiao commented Mar 19, 2025

root reason is that system is poor performance on risc-v machine. containerd get task status slow. there are some optimizations in containerd containerd/containerd@9e09bfb can you review this pr ? @AkihiroSuda thanks

@AkihiroSuda
Copy link
Member

ci failing

@ningmingxiao
Copy link
Contributor Author

done @AkihiroSuda

@AkihiroSuda AkihiroSuda added this to the v2.0.4 milestone Mar 19, 2025
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda merged commit fe82600 into containerd:main Mar 19, 2025
30 checks passed
@AkihiroSuda AkihiroSuda linked an issue Mar 20, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nerdctl ps time out show nothing
3 participants