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

Numan Siddique numans at ovn.org
Tue Aug 24 15:43:43 UTC 2021


On Mon, Aug 23, 2021 at 1:38 PM Odintsov Vladislav <VlOdintsov at croc.ru> wrote:
>
> Hi,
>
> I’m wonder if this patch series is interesting for the project.
> Should I wait for review or what is the process to add new functionality?

Hi Vladislav,

This is the present review queue -
https://patchwork.ozlabs.org/project/ovn/list/
and your patches are present.  Please expect some delay for the
reviews from my side
atleast.  Since they are submitted well within the cutoff time, these
patches should be
considered for the next release.

Thanks
Numan

>
> Thanks.
>
> Regards,
> Vladislav Odintsov
>
> On 17 Aug 2021, at 00:15, Vladislav Odintsov <odivlad at gmail.com<mailto:odivlad at gmail.com>> wrote:
>
> This patch series extends Logical Router's routing functionality.
> Now user may create multiple routing tables within a Logical Router
> and assign them to Logical Router Ports.
>
> Traffic coming from Logical Router Port with assigned route_table
> is checked against global routes if any (Logical_Router_Static_Routes
> whith empty route_table field), next against directly connected routes
> and then Logical_Router_Static_Routes with same route_table value as
> in Logical_Router_Port options:route_table field.
>
> This series doesn't have ddlog support yet. It will eventually be added
> once this series get reviewed.
>
> Vladislav Odintsov (3):
>  tests: remove strict check for table=N for N > 9
>  northd: support for RouteTables in LRs
>  utilities: update ovn-nbctl with RouteTables support
>
> northd/ovn-northd.8.xml |  63 ++++--
> northd/ovn-northd.c     | 198 ++++++++++++++---
> ovn-nb.ovsschema        |   5 +-
> ovn-nb.xml              |  30 +++
> tests/ovn-ic.at<http://ovn-ic.at>         |   4 +
> tests/ovn-nbctl.at<http://ovn-nbctl.at>      | 165 +++++++++++++-
> tests/ovn-northd.at<http://ovn-northd.at>     | 268 ++++++++++++++---------
> tests/ovn.at<http://ovn.at>            | 464 ++++++++++++++++++++++++++++++++++++++--
> utilities/ovn-nbctl.c   | 124 ++++++++++-
> 9 files changed, 1146 insertions(+), 175 deletions(-)
>
> --
> 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