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.

Sign up for GitHub

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

Jump to bottom

docs: kpr: DSR-Geneve with native-routing requires tunnelProtocol #30854

Merged
merged 1 commit into from Feb 20, 2024
Merged

docs: kpr: DSR-Geneve with native-routing requires tunnelProtocol #30854

merged 1 commit into from Feb 20, 2024

Conversation

julianwiedmann
Copy link
Member

Reflect the config change from #29051 in the kubeproxy-free docs for DSR-Geneve.

Fixes: #30845

@julianwiedmann
docs: kpr: DSR-Geneve with native-routing requires tunnelProtocol
f1eece2 
Reflect the config change from cilium#29051
in the kubeproxy-free docs for DSR-Geneve.

Fixes: cilium#30845
Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@julianwiedmann julianwiedmann added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. release-note/misc This PR makes changes that have no direct user impact. area/loadbalancing Impacts load-balancing and Kubernetes service implementations needs-backport/1.15 This PR / issue needs backporting to the v1.15 branch labels Feb 19, 2024
@julianwiedmann julianwiedmann requested review from a team as code owners February 19, 2024 15:26
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.15.2 Feb 19, 2024
@julianwiedmann julianwiedmann mentioned this pull request Feb 19, 2024
Closed
3 tasks
@julianwiedmann
Copy link
Member Author

/test

giorio94
giorio94 approved these changes Feb 19, 2024
View reviewed changes
Copy link
Member

@giorio94 giorio94 left a comment

Choose a reason for hiding this comment

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

Thanks!

ysksuzuki
ysksuzuki approved these changes Feb 20, 2024
View reviewed changes
Copy link
Member

@ysksuzuki ysksuzuki left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

learnitall
learnitall approved these changes Feb 20, 2024
View reviewed changes
Copy link
Contributor

@learnitall learnitall left a comment

Choose a reason for hiding this comment

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

Thanks!

@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 Feb 20, 2024
@julianwiedmann julianwiedmann added this pull request to the merge queue Feb 20, 2024
Merged via the queue into cilium:main with commit 8791007 Feb 20, 2024
60 checks passed
@julianwiedmann julianwiedmann deleted the 1.16-docs-geneve-dsr branch February 20, 2024 16:19
@YutaroHayakawa YutaroHayakawa mentioned this pull request Feb 27, 2024
Merged
9 tasks
@YutaroHayakawa YutaroHayakawa added backport-pending/1.15 The backport for Cilium 1.15.x for this PR is in progress. and removed needs-backport/1.15 This PR / issue needs backporting to the v1.15 branch labels Feb 27, 2024
@github-actions github-actions bot added backport-done/1.15 The backport for Cilium 1.15.x for this PR is done. and removed backport-pending/1.15 The backport for Cilium 1.15.x for this PR is in progress. labels Feb 28, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot removed this from Needs backport from main in 1.15.2 Feb 28, 2024
@jrajahalme jrajahalme mentioned this pull request Mar 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@borkmann borkmann borkmann approved these changes

@ysksuzuki ysksuzuki ysksuzuki approved these changes

@learnitall learnitall learnitall approved these changes

@giorio94 giorio94 giorio94 approved these changes

Assignees
No one assigned
Labels
area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. area/loadbalancing Impacts load-balancing and Kubernetes service implementations backport-done/1.15 The backport for Cilium 1.15.x for this PR is done. 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. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

KPR with DSR and Geneve dispatch, but can not work with native routing mode
6 participants
@julianwiedmann @borkmann @ysksuzuki @learnitall @giorio94 @YutaroHayakawa