[ovs-dev] [PATCH ovn v9 0/6] Add multiple routing tables support to Logical Routers
Numan Siddique
numans at ovn.org
Mon Nov 22 21:35:00 UTC 2021
On Fri, Nov 19, 2021 at 11:07 AM Vladislav Odintsov <odivlad at gmail.com> wrote:
>
> v8 -> v9:
> - Fix documentation errors.
> - Fix commit messages.
>
> 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 (6):
> northd: fix symmetric ECMP reply flows documentation
> 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.c | 2 +-
> lib/ovn-util.h | 3 +
> northd/northd.c | 192 ++++++++++---
> northd/ovn-northd.8.xml | 80 ++++--
> ovn-ic-sb.ovsschema | 8 +-
> ovn-ic-sb.xml | 30 ++
> ovn-nb.ovsschema | 5 +-
> ovn-nb.xml | 44 +++
> tests/ovn-ic.at | 550 +++++++++++++++++++++++++++++++++++++
> tests/ovn-nbctl.at | 196 ++++++++++++-
> tests/ovn-northd.at | 74 ++++-
> tests/ovn.at | 546 +++++++++++++++++++++++++++++++++++-
> utilities/ovn-nbctl.c | 138 +++++++++-
> 15 files changed, 2181 insertions(+), 283 deletions(-)
Thanks. I applied this patch series to the main branch.
Numan
>
> --
> 2.30.0
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
More information about the dev
mailing list