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.

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

Data plane VXLAN HLD - initial version #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

VladimirKuk
Copy link

No description provided.

@VladimirKuk VladimirKuk requested review from marvellgit and a team July 23, 2024 13:22
Copy link
Collaborator

@marvellgit marvellgit left a comment

Choose a reason for hiding this comment

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

vxlan coments

In small-scale datacenters, to reduce costs instead of BGP deployement, L3 forwarding could be based, either, on static configuration or usage of simpler protocol (such as OSPF or IS-IS).

Thus, in these cases, pure Layer2 VNI provides simple, flexible and cost-efficient solution.

Copy link
Collaborator

Choose a reason for hiding this comment

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

controller base vxlan network – supported by cisco / vmware
edge data centers

leaf vni {
mandatory true;
type stypes:vnid_type;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

this mean per leaf and map one remove vtep ?

Copy link
Author

@VladimirKuk VladimirKuk Jul 28, 2024

Choose a reason for hiding this comment

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

Yes, it is either this or list of vni's per remote VTEP


VXLAN_REMOTE_VTEP_TABLE|{{remote_vtep_name}}
"src_ip" : {{ipv4_address}}

Copy link
Collaborator

Choose a reason for hiding this comment

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

this can be refered as rempte ip ?

Copy link
Author

@VladimirKuk VladimirKuk Jul 28, 2024

Choose a reason for hiding this comment

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

you mean change the name from "src_ip" to "remote_ip" ? Done

In the current implementation P2MP tunnel creation flow exist with the exception of a bridgeport not created for P2MP tunnel. To support using P2MP tunnel for L2 purposes a bridge port is created for the P2MP tunnel object.

* Can/should it be supported in case of data plane tunnel ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

please explain the npon creating brg port , i think i does create on tunnel termination
if we support p2mp it means that user must configure every mac statically so i dont think this is needed

Copy link
Author

Choose a reason for hiding this comment

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

Brigdeport explanation is taken from old HLD by mistake.
I rephrased the section, please changed text.

@VladimirKuk VladimirKuk requested a review from marvellgit July 29, 2024 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants