[ovs-dev] [PATCH ovn v9 0/6] Add multiple routing tables support to Logical Routers
VlOdintsov at croc.ru
Tue Nov 23 12:54:44 UTC 2021
> On 23 Nov 2021, at 00:35, Numan Siddique <numans at ovn.org> wrote:
> 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
>> - 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`
>> and `ic: process only local port_bindings`
>> 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.
>> dev mailing list
>> dev at openvswitch.org
More information about the dev