Skip to content

docs: add note on sctp for kube-proxy replacement as known limitation#12433

Merged
borkmann merged 1 commit intomasterfrom
pr/docs-sctp-kube-proxy
Jul 6, 2020
Merged

docs: add note on sctp for kube-proxy replacement as known limitation#12433
borkmann merged 1 commit intomasterfrom
pr/docs-sctp-kube-proxy

Conversation

@borkmann
Copy link
Copy Markdown
Member

@borkmann borkmann commented Jul 6, 2020

Noticed we didn't mention anything about it yet, so state it explicitly here.

Signed-off-by: Daniel Borkmann daniel@iogearbox.net

Noticed we didn't mention anything about it yet, so state it explicitly here.

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
@borkmann borkmann added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. release-note/misc This PR makes changes that have no direct user impact. area/kube-proxy-free labels Jul 6, 2020
@borkmann borkmann requested a review from brb July 6, 2020 14:04
@borkmann borkmann requested a review from a team as a code owner July 6, 2020 14:04
Copy link
Copy Markdown
Member

@brb brb left a comment

Choose a reason for hiding this comment

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

LGTM. OT: does Cilium + kube-proxy work for SCTP?

@borkmann
Copy link
Copy Markdown
Member Author

borkmann commented Jul 6, 2020

LGTM. OT: does Cilium + kube-proxy work for SCTP?

Most likely not, but its in alpha anyway (but also since our ClusterIP handling is not SCTP aware either) ... https://kubernetes.io/docs/concepts/services-networking/service/#protocol-support . And given how buggy the SCTP kernel code is I wouldn't recommend to use it generally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants