[ovs-dev] [PATCH ovn v6 0/4] Add multiple routing tables support to Logical Routers

Vladislav Odintsov odivlad at gmail.com
Tue Oct 12 13:26:18 UTC 2021


Hi Han,

could you please take a look on this when you have time?
Thanks.

Regards,
Vladislav Odintsov

> On 5 Oct 2021, at 23:24, Vladislav Odintsov <odivlad at gmail.com> wrote:
> 
> 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 (4):
>  ic: process only local port_bindings
>  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             | 576 ++++++++++++++++++++++++++--------------
> northd/northd.c         | 159 ++++++++---
> northd/ovn-northd.8.xml |  63 ++++-
> ovn-ic-sb.ovsschema     |   5 +-
> ovn-ic-sb.xml           |  18 ++
> ovn-nb.ovsschema        |   5 +-
> ovn-nb.xml              |  30 +++
> tests/ovn-ic.at         | 493 ++++++++++++++++++++++++++++++++++
> tests/ovn-nbctl.at      | 196 +++++++++++++-
> tests/ovn-northd.at     |  76 +++++-
> tests/ovn.at            | 441 +++++++++++++++++++++++++++++-
> utilities/ovn-nbctl.c   | 138 +++++++++-
> 13 files changed, 1937 insertions(+), 267 deletions(-)
> 
> -- 
> 2.30.0
> 



More information about the dev mailing list