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

The sandbox.yaml reference in the containerd-env-setup.md document has issues #1496

Closed
whygyc opened this issue Nov 29, 2023 · 3 comments · Fixed by #1499
Closed

The sandbox.yaml reference in the containerd-env-setup.md document has issues #1496

whygyc opened this issue Nov 29, 2023 · 3 comments · Fixed by #1499

Comments

@whygyc
Copy link
Contributor

whygyc commented Nov 29, 2023

Related issues: kubernetes-sigs/cri-tools#1273

The nydus-sandbox.yaml does not specify a uid which causes the crictl rmp to fail

metadata:
  attempt: 1
  name: nydus-sandbox
  namespace: default
log_directory: /tmp
linux:
  security_context:
    namespace_options:
      network: 2

As a test example, uid can be specified as an arbitrary string, like:

metadata:
  attempt: 1
  name: nydus-sandbox
  namespace: default
  uid: nydus-sandbox-test

This issue will be fixed in the new version, but it remains a problem for those using slightly earlier versions of crictl.

@imeoer
Copy link
Collaborator

imeoer commented Nov 29, 2023

Thanks for reporting, cloud you create a PR to fix it? thanks!

@whygyc
Copy link
Contributor Author

whygyc commented Nov 29, 2023

https://github.com/dragonflyoss/nydus/blob/master/docs/containerd-env-setup.md

If you have encountered a similar issue, you can use ctr -n k8s.io c ls to view the same ID and use ctr -n k8s.io c rm <id> to delete it. After restarting the containerd service, it will no longer appear in crictl pods.

# crictl rmp 2aec11d869854
getting sandbox status of pod "2aec11d869854": metadata.Name, metadata.Namespace or metadata.Uid is not in metadata "&PodSandboxMetadata{Name:nydus-sandbox,Uid:,Namespace:default,Attempt:1,}"

@whygyc
Copy link
Contributor Author

whygyc commented Nov 29, 2023

Thanks for reporting, cloud you create a PR to fix it? thanks!

ok, I will do it.

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 a pull request may close this issue.

2 participants