Changwei Ge

When this done, we'd better add a note to README to tell that nydus-snapshotter implements all the snapshots APIs

Thanks for considering this. Frankly speaking, the test for it is fragile.

Are multiple shared nydusd daemons running simultaneously or only one nydusd is allowed for shared mode?

SGTM. My only suggestion is to add the designated KV to annotations rather than POD label since the label is used for node selection and affinity.

> duplicate with #8402. I think we can focus on existing pr and push forward. Make sense  :-) I didn't notice that another PR similar to https://github.com/containerd/containerd/pull/4785 was raised before....

> > If we don't validate the task response, the sandbox task pid is ZERO which is later passed to the business container and requests it to join the namespace...

> Hi guys we are encountering similar issue, in our case, shim will indeterminably fail. (works fine for sandbox and init container but fails for main container) > > my...

> maybe image is broken because of unexpected poweroff .try [#10284](https://github.com/containerd/containerd/pull/10284) or [#9401](https://github.com/containerd/containerd/pull/9401) to enable sync. The mentioned PRs are addressing data loss for snapshotter, which stores unpacked data whereas...

Generally, it can hardly fail to delete blob cache stuff. Can you see the particular error when it failed? Even we migrate a synchronously way to remove blob cache, it...

@lukasmrtvy > @changweige can we make the nydus-snapshotter-config optional? or make it available as a string in values.yaml > > for example ( values.yaml ): > > ``` > config:...