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

Correct cni path in k3s.rst for rancher desktop #27702

Merged
merged 1 commit into from
Aug 25, 2023

Conversation

RichardoC
Copy link
Contributor

@RichardoC RichardoC commented Aug 25, 2023

Correct cni path in k3s installation documentation for rancher desktop

Signed-off-by: Richard Tweed <RichardoC@users.noreply.github.com>
@RichardoC RichardoC requested a review from a team as a code owner August 25, 2023 11:58
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 25, 2023
@github-actions github-actions bot added the kind/community-contribution This was a contribution made by a community member. label Aug 25, 2023
@RichardoC
Copy link
Contributor Author

RichardoC commented Aug 25, 2023

This was discovered by following https://docs.cilium.io/en/stable/gettingstarted/k8s-install-default/#k8s-install-quick on an M1 macbook, with Rancher Desktop 1.9.1

Failure message when try to create pods post installation if you don't add this flag is below

$ kubectl get pods
NAME                                     READY   STATUS              RESTARTS        AGE
somepod-77ccd57875-6pr7s                 0/1     ContainerCreating   0               30s
$ kubectl describe pods
.....
Events:
  Type     Reason                  Age                    From               Message
  ----     ------                  ----                   ----               -------
  Warning  FailedCreatePodSandBox  8m9s                   kubelet            Failed to create pod sandbox: rpc error: code = Unknown desc = failed to setup network for sandbox "77b80d18d8cee329081f8917125494de0a711c241bc38817835b854b8a6014f3": plugin type="cilium-cni" failed (add): failed to find plugin "cilium-cni" in path [/usr/libexec/cni]
  Normal   SandboxChanged          2m58s (x25 over 8m9s)  kubelet            Pod sandbox changed, it will be killed and re-created.

@lambdanis lambdanis self-assigned this Aug 25, 2023
@lambdanis lambdanis added the release-note/misc This PR makes changes that have no direct user impact. label Aug 25, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 25, 2023
@lambdanis
Copy link
Contributor

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 25, 2023
@borkmann borkmann added the needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch label Aug 25, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.14.2 Aug 25, 2023
@borkmann borkmann merged commit 0481d5f into cilium:main Aug 25, 2023
57 checks passed
@pippolo84 pippolo84 mentioned this pull request Aug 28, 2023
9 tasks
@pippolo84 pippolo84 added backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. and removed needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch labels Aug 28, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from main to Backport pending to v1.14 in 1.14.2 Aug 28, 2023
@michi-covalent michi-covalent added backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. and removed backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. labels Sep 9, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.14 to Backport done to v1.14 in 1.14.2 Sep 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. kind/community-contribution This was a contribution made by a community member. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.14.2
Backport done to v1.14
Development

Successfully merging this pull request may close these issues.

None yet

5 participants