containerd / nydus-snapshotter

A containerd snapshotter with data deduplication and lazy loading in P2P fashion
https://nydus.dev/
Apache License 2.0
156 stars 88 forks source link

go.mod: github.com/containerd/containerd v1.7.7, containerd/nri v0.4.0, switch to github.com/containerd/log module #598

Closed thaJeztah closed 1 week ago

thaJeztah commented 1 week ago

go.mod: github.com/containerd/containerd v1.7.7, containerd/nri v0.4.0

Update to the first release of containerd to alias the log package to the new containerd/log module.

This update of the containerd module also comes with an updated version of github.com/containerd/nr, which has some breaking changes in the interface definitions.

the optimizer-nri-plugin and prefetchfilees-nri-pugin pacakges had to be updated due to containerd/nri@9d86150 changing the interface to require a context to be passed.

Full diffs:

switch to github.com/containerd/log module

containerd 1.7.7 and up alias the log package to the new module, and deprecate the package.

This patch:

There is one change in behavior in the nri-plugins, which now use the default format as used by the containerd log module.

thaJeztah commented 1 week ago

@imeoer ptal 🤗

imeoer commented 1 week ago

LGTM, thanks for the careful work! :)

thaJeztah commented 1 week ago

@imeoer thanks! I also move https://github.com/containerd/nydus-snapshotter/pull/599 out of draft, but it looks like there's an issue with OTEL 🤔