Skip to content
This repository was archived by the owner on Jun 20, 2024. It is now read-only.

Update docs to include issue with containerd versions <1.6.5 #3946

Merged
merged 2 commits into from
Jun 9, 2022

Conversation

rajch
Copy link
Contributor

@rajch rajch commented Jun 8, 2022

If a kubernetes cluster uses the containerd cri, and containerd is
at version 1.6.0 through 1.6.4, weave does not work. This doc
update highlights this and provides a solution, which is to
upgrade containerd to 1.6.6.

The following sequence of issues and PRs were used to create
this doc:

#3936
containerd/containerd#6921
containernetworking/cni#895
containernetworking/cni#896
containernetworking/cni#899
containerd/containerd#7009

This addresses #3936, and partially #3942 and the discussion
in #3939.

@kingdonb, since this is a user-facing document, I did not
provided very detailed technical information; just the problem
and solution. Hope that's okay.

If a kubernetes cluster uses the containerd cri, and containerd is
at version 1.6.0 through 1.6.4, weave does not work. This doc
update highlights this and provides a solution, which is to
upgrade containerd.
@kingdonb kingdonb self-requested a review June 8, 2022 13:53
@kingdonb
Copy link
Contributor

kingdonb commented Jun 8, 2022

Thank you, I will take a look at this as soon as GitOpsDays is over 🎉

(https://gitopsdays.com - for anyone who is interested, we have this two day conference kicking off in a few minutes)

This looks like a terrific level of detail, I'm mainly checking to be sure I know how the docs are published before I merge it

Copy link
Contributor

@kingdonb kingdonb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some suggested changes

@kingdonb
Copy link
Contributor

kingdonb commented Jun 9, 2022

OK, this is either going to land here after it merges:

https://www.weave.works/docs/net/latest/kubernetes/kube-addon/
or we will find there are additional steps, and I need to read more docs 👍 let's find out

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants