[ovs-dev,v9,0/2] OVN: Distributed Virtual Router for Vlan Backed Networks
mbox series

Message ID 1559175728-127062-1-git-send-email-ankur.sharma@nutanix.com
Headers show
Series
  • OVN: Distributed Virtual Router for Vlan Backed Networks
Related show

Message

Ankur Sharma May 30, 2019, 12:20 a.m. UTC
This series is about enhancing the logical router functionality in OVN to work
with vlan backed logical switches.

Intial proposal was discused here:
[1] https://mail.openvswitch.org/pipermail/ovs-dev/2018-October/353066.html
[2] https://docs.google.com/document/d/1uoQH478wM1OZ16HrxzbOUvk5LvFnfNEWbkPT6Zmm9OU/edit?usp=sharing

This series covers following:
a. L2:
   Associate a type with logical switches. Type value could be vlan or bridged.

b. L3 E-W:
   In the absence of encapsulation, we cannot use router port mac as source mac
   (since it is distributed), hence replace the same with a chassis unique mac.

c. L3 N-S:
   Use gateway-chassis construct to respond to ARP requests for router port,
   so that it becomes entry point for all chassis bound traffic coming from
   "external" network.
   Some additional changes, like no need to redirect south to north traffic
   in the absence of NAT etc.

This series does not cover following:
(will be sent out for review in follow up series once this series is reviewed/committed)
a. Network Address Translation.
b. Ensuring VMs mac is learnt in underlay network to avoid flooding of L3 flows.

v8 -> v9
----
  * Added more unit tests.
  * Fix the mistake of adding cr-lrp checks for IPv6 rather than V4.

v7 -> v8
----
  * Handled Numan's review comments.

v6 -> v7
----
  * Added packet flow details in ovn-architecture.

v5 -> v6
----
  * Handled Numan's review comments.


v4 -> v5
----
  * Changed the network_type value to "bridged" instead of "vlan".

v3 -> v4
----
  * Improved documentation and commit message.
  * Converted the patchset to 2 patches.
  * Changes in nbctl to NOT set to network_type during logical switch create.
  * Fixed 3 space indentation.

v2 -> v3
-----
  * Rebased with TOT.

v1 -> v2
------
  * Fixed unit tests and review comments.

Ankur Sharma (2):
  OVN: Enable E-W Traffic, Vlan backed DVR
  OVN: Enable N-S Traffic, Vlan backed DVR

 ovn/controller/binding.c            |  12 +-
 ovn/controller/chassis.c            |  64 ++-
 ovn/controller/chassis.h            |   4 +
 ovn/controller/ovn-controller.8.xml |  10 +
 ovn/controller/ovn-controller.c     |   4 +-
 ovn/controller/ovn-controller.h     |   5 +-
 ovn/controller/physical.c           | 113 +++++
 ovn/controller/pinctrl.c            | 205 ++++++--
 ovn/controller/pinctrl.h            |   6 +
 ovn/lib/ovn-util.c                  |  31 ++
 ovn/lib/ovn-util.h                  |   6 +
 ovn/northd/ovn-northd.c             |  81 +++-
 ovn/ovn-architecture.7.xml          | 111 ++++-
 ovn/ovn-nb.ovsschema                |  10 +-
 ovn/ovn-nb.xml                      |  19 +
 ovn/ovn-sb.xml                      |  15 +
 ovn/utilities/ovn-nbctl.c           |  49 +-
 tests/ovn-nbctl.at                  |  48 +-
 tests/ovn-northd.at                 |  22 +
 tests/ovn.at                        | 927 ++++++++++++++++++++++++++++++++++++
 20 files changed, 1669 insertions(+), 73 deletions(-)