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
Expanded Cilium deployment documentation #5012
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️ , Awesome, this is really comprehensive a small typo and this is good.
@SixFive7 can you sign off the commit with |
All commits should be signed now except those done through the web ui by clicking the "accept suggested changes" button. Turns out this is not yet possible: todogroup/gh-issues#50 |
Update cilium docs Co-authored-by: Jori Huisman <6650122+SixFive7@users.noreply.github.com> supercededs: siderolabs#5012 Signed-off-by: Noel Georgi <git@frezbo.dev>
Update cilium docs Co-authored-by: Jori Huisman <6650122+SixFive7@users.noreply.github.com> supersedes: siderolabs#5012 Signed-off-by: Noel Georgi <git@frezbo.dev>
/ok-to-test |
Expanded Cilium deployment documentation Co-authored-by: Noel Georgi <git@frezbo.dev> Signed-off-by: Noel Georgi <git@frezbo.dev>
/m |
Pull Request
What?
Expanded a little on the Talos specific Cilium deployment details.
Why?
There are some gotchas when deploying Cilium onto Talos that takes time to figure out if not documented.
Related issues mentioned in the documentation
#4109 GCP ILB support / support scope local routes to be configured
#4212 add Node.PodCIDRs to the k8s nodeadress filter
#4654 Kernel default values (sysctl)
#4756 Cilium Documentation Generates Clusters Without a CNI
#4836 Kubespan and Cilium compatiblity: etcd is failing
#4863 Cilium on talos "aoe: packet could not be sent on *. consider increasing tx_queue_len"
Acceptance
Please use the following checklist:
make conformance
)make fmt
)make lint
)make docs
)make unit-tests
)This change is