Modify networking to enable integration with K3s #3252
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
eriknordmark
reviewed
Jun 1, 2023
eriknordmark
reviewed
Jun 1, 2023
eriknordmark
reviewed
Jun 1, 2023
eriknordmark
reviewed
Jun 1, 2023
eriknordmark
approved these changes
Jun 1, 2023
uncleDecart
approved these changes
Jun 2, 2023
milan-zededa
force-pushed
the
k3s-networking
branch
from
bbf5b95 to
6dc8d8d
Compare
I could squash, but I didn't want the Naiming's contribution to disappear, hence I kept his commits :) |
milan-zededa
force-pushed
the
k3s-networking
branch
3 times, most recently
from
6cde70b to
54df19b
Compare
milan-zededa
force-pushed
the
k3s-networking
branch
from
54df19b to
483f0bd
Compare
|
Resolved merge conflicts |
|
Seems like we have some issues in master causing ztests failures. Let's wait until those are resolved and then retest and merge this PR. |
|
@milan-zededa Makes sense to rebase this on master so we can get reasonable test results. |
Signed-off-by: Naiming Shen <naiming@zededa.com>
Signed-off-by: Naiming Shen <naiming@zededa.com>
This enables communication between K3s API server (listening on a port IP address) and other K3s components (listening on IPs from the pod subnet). Signed-off-by: Milan Lenco <milan@zededa.com>
milan-zededa
force-pushed
the
k3s-networking
branch
from
483f0bd to
3c31e47
Compare
@eriknordmark Done, rebased. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is related to the integration of EVE with K3s (#3210) and covers the networking part.
For now we will support specifically only the (default) Flannel CNI.
It turns out that it is necessary to propagate routes created for Flannel's bridge
cni0into the per-port routing tables (managed by NIM).