mbox series

[ovs-dev,v8,0/5] Add multiple routing tables support to Logical Routers

Message ID 20211113094353.17690-1-odivlad@gmail.com
Headers show
Series Add multiple routing tables support to Logical Routers | expand

Message

Vladislav Odintsov Nov. 13, 2021, 9:43 a.m. UTC 
v7 -> v8:
  - Updated manpages with and commit messages according to latest changes.
  - Small fixes.

v6 -> v7:
  - Addressed Han's comments regarding saving same behaviour for route-tables
    static routes as for static routes from 'global' table.
    To address this 2 new patches were added (1, 2).
  - "global" table terminology was replaced with '<main>'.

v5 -> v6:
  - 2 memory leak bugs were fixed in ovn-ic code, which were introduced in
    previous versions of #3 patch.

v4 -> v5:
  - Addressed Numan's review comments.

v3 -> v4:
  - Minor logging typo fixes.
  - Added patch with ovn-ic routes learning bugfix.

v2 -> v3:
  - Rebased on split northd changes.
  - Replaced route_tables HMAP with SIMAP as Numan suggested.
  - This series stil doesn't have ddlog support yet.
    It will take too much time for me to deal to ddlog language and specifics.
    Help with ddlog implementation wanted.

v1 -> v2:
  - First patch of v1 patch series was applied, but new tests for new feature
    were added with strict table number check. Update this tests to be table
    number-independent.
  - Squash pathes for northd and utilities as tests don't pass without latter.
  - Add support for OVN IC routing table in routes advertisement/learning.
  - Patches `ic: remove port_binding on ts deletion`
    https://patchwork.ozlabs.org/project/ovn/patch/20210824184442.35063-1-odivlad@gmail.com/
    and `ic: process only local port_bindings`
    https://patchwork.ozlabs.org/project/ovn/patch/20210830195707.98529-1-odivlad@gmail.com/
    were already sent to list separately, but other changes are based on them
    so they're included.
    Once those patches are accepts, I can drop them from this series.
  - Added NEWS item.
  - Added myself to authors list.

Vladislav Odintsov (5):
  ic: maintain route origin - connected/static
  northd: make connected routes have higher priority than static
  northd,utils: support for RouteTables in LRs
  ic: add support for routing tables in adv/learn routes
  ic: don't learn routes which have local GW

 NEWS                    |   4 +
 ic/ovn-ic.c             | 592 ++++++++++++++++++++++++++--------------
 lib/ovn-util.h          |   3 +
 northd/northd.c         | 192 ++++++++++---
 northd/ovn-northd.8.xml |  94 +++++--
 ovn-ic-sb.ovsschema     |   8 +-
 ovn-ic-sb.xml           |  30 ++
 ovn-nb.ovsschema        |   5 +-
 ovn-nb.xml              |  30 ++
 tests/ovn-ic.at         | 550 +++++++++++++++++++++++++++++++++++++
 tests/ovn-nbctl.at      | 196 ++++++++++++-
 tests/ovn-northd.at     |  74 ++++-
 tests/ovn.at            | 546 +++++++++++++++++++++++++++++++++++-
 utilities/ovn-nbctl.c   | 138 +++++++++-
 14 files changed, 2174 insertions(+), 288 deletions(-)