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

[Vladislav Odintsov]

Thanks Numan for the information.

Regards,
Vladislav Odintsov

> On 24 Aug 2021, at 18:43, Numan Siddique  wrote:
> 
> On Mon, Aug 23, 2021 at 1:38 PM Odintsov Vladislav  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 
>> mailto:odiv...@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 |   4 +
>> tests/ovn-nbctl.at  | 165 +-
>> tests/ovn-northd.at | 268 ++-
>> tests/ovn.at| 464 
>> ++--
>> utilities/ovn-nbctl.c   | 124 ++-
>> 9 files changed, 1146 insertions(+), 175 deletions(-)
>> 
>> --
>> 2.30.0
>> 
>> 
>> ___
>> dev mailing list
>> d...@openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

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

[Numan Siddique]

On Mon, Aug 23, 2021 at 1:38 PM Odintsov Vladislav  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 
> mailto:odiv...@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 |   4 +
> tests/ovn-nbctl.at  | 165 +-
> tests/ovn-northd.at | 268 ++-
> tests/ovn.at| 464 
> ++--
> utilities/ovn-nbctl.c   | 124 ++-
> 9 files changed, 1146 insertions(+), 175 deletions(-)
>
> --
> 2.30.0
>
>
> ___
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

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

[Odintsov Vladislav]

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?

Thanks.

Regards,
Vladislav Odintsov

On 17 Aug 2021, at 00:15, Vladislav Odintsov 
mailto:odiv...@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 |   4 +
tests/ovn-nbctl.at  | 165 +-
tests/ovn-northd.at | 268 ++-
tests/ovn.at| 464 
++--
utilities/ovn-nbctl.c   | 124 ++-
9 files changed, 1146 insertions(+), 175 deletions(-)

--
2.30.0


___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

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

[Vladislav Odintsov]

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 |   4 +
 tests/ovn-nbctl.at  | 165 +-
 tests/ovn-northd.at | 268 ++-
 tests/ovn.at| 464 ++--
 utilities/ovn-nbctl.c   | 124 ++-
 9 files changed, 1146 insertions(+), 175 deletions(-)

-- 
2.30.0

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev