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

Document comptable Containerd versions in README.md #6

Closed
jiangliu opened this issue Dec 13, 2021 · 3 comments · Fixed by #37
Closed

Document comptable Containerd versions in README.md #6

jiangliu opened this issue Dec 13, 2021 · 3 comments · Fixed by #37

Comments

@jiangliu
Copy link
Contributor

Containerd is still evolving quickly, interfaces and behaviors may change between Cotnainerd versions. It would be great to document compatible Containerd versions for each version of rust crates in the README.md.

@lifupan
Copy link

lifupan commented Dec 13, 2021

Yeah,it's better to release the extensions version the same to the latest compatible containerd version, thus the extension users can easily know that their programs compatible with which containerd version.

@mxpv
Copy link
Member

mxpv commented Dec 14, 2021

We usually don't do changes that break compatibility but rather deprecate things.
Shim runtime v2 expected to work with newer versions of containerd.
(we have an issue to add CI tests to track binary compatibility - containerd/containerd#3757).

So I'm not sure there is strong need to keep up with containerd releases.
I'd rather do (somewhere on my list) integration tests for the rust extensions. I believe we can have a matrix of stable containerd versions similarly to how it's done in nerdctl (example https://github.com/containerd/nerdctl/runs/4518724866) to make sure no things broken in containerd releases we currently support.

@jiangliu
Copy link
Contributor Author

It's a great idea to setup CI tests to ensure compatibility between shim and Containerd.
The goal is to guarantee compatibility:)

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.

3 participants