Skip to content

VNet/Vxlan HLD#240

Merged
prsunny merged 21 commits intosonic-net:gh-pagesfrom
prsunny:gh-pages
Oct 18, 2018
Merged

VNet/Vxlan HLD#240
prsunny merged 21 commits intosonic-net:gh-pagesfrom
prsunny:gh-pages

Conversation

@prsunny
Copy link
Copy Markdown
Contributor

@prsunny prsunny commented Aug 30, 2018

High level design for Sonic Vxlan

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copy link
Copy Markdown

@pavel-shirshov pavel-shirshov Sep 4, 2018

Choose a reason for hiding this comment

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

Probably it's better to have one name of the attribute to refer to vni? I'd put vni here.

Copy link
Copy Markdown
Contributor

@jipanyang jipanyang left a comment

Choose a reason for hiding this comment

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

Is there case that vxlan packet gets trapped to hostif?
If the answer is yes, what will the packet format be like and how to handle it?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For the schemas defined here, are they for appDB or configDB?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

better to specify this clearly.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I noticed that VNI to VLAN map is added to config DB, while VNI to virtual router map is added to app DB.
Is there a reason for separating these two types of tunnel maps?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, the idea is to have VxlanOrch only deal with Tunnel configurations (Encap/Decap etc). The peering must be required only by VRF/RouterOrch.

@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented Sep 5, 2018

for vxlan decap flow, how you decide which pipeline the packet take after decap, l2 or l3?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

the comment should be the destination address

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Only one vxlan_tunnel per vnet?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

What will be the total number of routes to be supported ? Can that be added to this table as well. - thx

@prsunny prsunny changed the title Vxlan HLD VNet/Vxlan HLD Sep 21, 2018
@prsunny prsunny merged commit 0d8da56 into sonic-net:gh-pages Oct 18, 2018
lguohan pushed a commit that referenced this pull request Nov 28, 2018
High level design for Sonic Vxlan
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.

7 participants