-
Notifications
You must be signed in to change notification settings - Fork 103
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
Kubelet log has error when use default root dir #288
Comments
Does it mean we must use |
Yes, if the It seems that the nydus snapshotter's root directory must be under containerd's root to avoid these errors. But containerd |
Seems the snapshotter root directory is passed to Kubelet for components like cAdvisor to collect statics of POD. The POD eviction based on imageFS does not need it. The disk usage is reported by nydus-snapshotter to containerd. |
We still need to ensure no error messages are thrown, right? Although maybe it doesn't have any effect. |
I think fixing this on the containerd side will be ideal. |
I'm trying to use helm install the nydus-snapshotter, I got this error too. I can't find any place to modify the root option, how can i fix it? |
Based on what version of nydus-snapshotter you are working on, if it is a self-built nydus-snapshotter from the main branch, it can be tweaked in its configuration file. If it is 0.5 or below of nydus-snapshotter, nydus-snapshotter's root dir can be changed by CLI parameter |
Thanks for your reply, The default value of the tag in the helm chart I use is v0.4.0. |
Yes. 0.5.1 can work |
I rebuilt the image of v0.5.1, modified the root path, and there is no problem in running it, but when creating a pod, the following error will be reported.
I checked the /var/lib/containerd/io.containerd.content.v1.content/blobs/sha256 path and there is no this file, I guess the metadata is wrong somewhere? |
Is the image that you are running OCI format or nydus format? Looks like this is an error from Containerd. Can you ensure that containerd is using nydus-snapshotter |
@Riverdd If you have a boltdb inspector like https://github.com/br0xen/boltbrowser, please try to scan nydus-snapshotter's snapshots DB located at its |
I am using the sandbox image in OCI format, can I use the image in OCI format when the containerd snapshotter is configured as nydus? |
Yes. Nydus-snasphotter is compatible with OCI images, which means OCI image can be unpacked to the snapshots created by nydus-snapshotter. Aha, I guess the reason of your issue is that the ROOT dir is changed, where stores nydus-snapshotter snapshots DB |
Looks right, but the old path has been deleted by me. |
Hi @Riverdd, is it convenient for you to join DingTalk group |
OK, let's change the channel. |
Related fixup on containerd: containerd/containerd#10127 |
But we'd better still set snapshotter's default root path to |
The log(combined contaienrd and kubelet):
Continerd returns a fixed-format directory for snapshotters:
The text was updated successfully, but these errors were encountered: