enhancement: internal iBGP route reflector on control plane nodes#218
Open
qinqon wants to merge 1 commit into
Open
enhancement: internal iBGP route reflector on control plane nodes#218qinqon wants to merge 1 commit into
qinqon wants to merge 1 commit into
Conversation
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.
/kind design
What this PR does / why we need it:
Adds the enhancement document for an internal iBGP Route Reflector running on control plane nodes.
In cloud environments where the ToR does not support EVPN, east/west traffic cannot be distributed via the fabric. This enhancement adds an internal RR so EVPN routes are reflected between router pods and VXLAN data plane goes directly node-to-node, avoiding the ToR.
Design highlights:
runOnMaster:true: reuse existing router pods + hostcontroller on CP, add RR config viaRawFRRConfigfalse: deploy additional RR-only router pods on CP with a separate Underlay CR (NIC + ASN, no ToR, no EVPN data plane)bgp listen rangeon RR side for accepting dynamic clientsallowas-inonly on eBGP neighbors (template conditional fix needed)underlay-ipsannotation, no hostcontroller cache extensionSpecial notes for your reviewer:
This is the enhancement document only, no code changes.
Release note: