Allow specification of vxlan src/dst ports

ligato / vpp-agent

⚡️ Control plane management agent for FD.io's VPP

https://docs.ligato.io/

Apache License 2.0

247 stars 127 forks source link

Allow specification of vxlan src/dst ports #1020

Open edwarnicke opened 5 years ago

edwarnicke commented 5 years ago

I realize this will also require vpp to add support for non-standard vxlan src/dst ports, but most other vxlan implementations allow this, and its important in circumstances where one or the other side of a tunnel cannot use the standard ports.

ondrej-fabry commented 5 years ago

Attaching links for reference:

NB Model: https://github.com/ligato/vpp-agent/blob/219476d3c10dfb8ed4457f817c5cfa8f9435420e/plugins/vpp/model/interfaces/interfaces.proto#L78-L84

Binapi Message: https://github.com/ligato/vpp-agent/blob/219476d3c10dfb8ed4457f817c5cfa8f9435420e/plugins/vpp/binapi/vxlan/vxlan.ba.go#L82-L92

Label this as blocked until it's implemented in VPP.

edwarnicke commented 5 years ago

As mentioned, I do know its blocked till implemented in vpp :)

Just wanted to get it on record here so you guys got some forwarning :)

ondrej-fabry commented 5 years ago

Copy that. My comment was only meant as additional reference for others.