Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-11-13 Thread Odintsov Vladislav
Thanks Numan, I’ve forgotten to update description according to new changes. A new patch version v8 was submitted: https://patchwork.ozlabs.org/project/ovn/cover/2023094353.17690-1-odiv...@gmail.com/ Please check it out. Regards, Vladislav Odintsov On 12 Nov 2021, at 23:07, Numan Siddique m

Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-11-12 Thread Numan Siddique
On Thu, Nov 11, 2021 at 2:18 PM Odintsov Vladislav wrote: > > Hi Han, Numan, > > I’ve posted a new version of this series [1] and addressed your comments and > suggestions. > I’ll appreciate if you can take a look on this please and I’d be happy if > this can be included in next OVN release. > >

Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-11-11 Thread Odintsov Vladislav
Hi Han, Numan, I’ve posted a new version of this series [1] and addressed your comments and suggestions. I’ll appreciate if you can take a look on this please and I’d be happy if this can be included in next OVN release. Thanks. 1: https://patchwork.ozlabs.org/project/ovn/cover/2021191306

Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-10-20 Thread Han Zhou
On Tue, Oct 19, 2021 at 3:21 PM Numan Siddique wrote: > > On Tue, Oct 19, 2021 at 3:28 AM Han Zhou wrote: > > > > On Mon, Oct 18, 2021 at 6:35 AM Odintsov Vladislav > > wrote: > > > > > > > > > > > > regards, > > > Vladislav Odintsov > > > > > > > On 16 Oct 2021, at 03:20, Han Zhou wrote: > > >

Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-10-19 Thread Numan Siddique
On Tue, Oct 19, 2021 at 3:28 AM Han Zhou wrote: > > On Mon, Oct 18, 2021 at 6:35 AM Odintsov Vladislav > wrote: > > > > > > > > regards, > > Vladislav Odintsov > > > > > On 16 Oct 2021, at 03:20, Han Zhou wrote: > > > > > > On Fri, Oct 15, 2021 at 2:36 AM Vladislav Odintsov > > > wrote: > > >

Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-10-19 Thread Han Zhou
On Mon, Oct 18, 2021 at 6:35 AM Odintsov Vladislav wrote: > > > > regards, > Vladislav Odintsov > > > On 16 Oct 2021, at 03:20, Han Zhou wrote: > > > > On Fri, Oct 15, 2021 at 2:36 AM Vladislav Odintsov > > wrote: > > > >> > >> > >> Regards, > >> Vladislav Odintsov > >> > >> On 15 Oct 2021, at

Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-10-15 Thread Han Zhou
On Fri, Oct 15, 2021 at 2:36 AM Vladislav Odintsov wrote: > > > Regards, > Vladislav Odintsov > > On 15 Oct 2021, at 08:42, Han Zhou wrote: > > On Thu, Oct 14, 2021 at 12:58 AM Vladislav Odintsov > wrote: > > > Hi Han, > > Thanks for the review. > > Regards, > Vladislav Odintsov > > On 14 Oct 2

Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-10-15 Thread Vladislav Odintsov
Regards, Vladislav Odintsov > On 15 Oct 2021, at 08:42, Han Zhou wrote: > > On Thu, Oct 14, 2021 at 12:58 AM Vladislav Odintsov > wrote: >> >> Hi Han, >> >> Thanks for the review. >> >> Regards, >> Vladislav Odintsov >> >> On 14 Oct 2021, at 08:13, Han Zhou wrote: >> >> >> >> On Tue,

Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-10-14 Thread Han Zhou
On Thu, Oct 14, 2021 at 12:58 AM Vladislav Odintsov wrote: > > Hi Han, > > Thanks for the review. > > Regards, > Vladislav Odintsov > > On 14 Oct 2021, at 08:13, Han Zhou wrote: > > > > On Tue, Oct 5, 2021 at 1:26 PM Vladislav Odintsov wrote: > > > > This patch extends Logical Router's routing f

Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-10-14 Thread Vladislav Odintsov
Hi Han, Thanks for the review. Regards, Vladislav Odintsov > On 14 Oct 2021, at 08:13, Han Zhou wrote: > > > > On Tue, Oct 5, 2021 at 1:26 PM Vladislav Odintsov > wrote: > > > > This patch extends Logical Router's routing functionality. > > Now user may create mult

Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-10-13 Thread Han Zhou
On Tue, Oct 5, 2021 at 1:26 PM Vladislav Odintsov wrote: > > This patch 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_t

Re: [ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-10-07 Thread 0-day Robot
Bleep bloop. Greetings Vladislav Odintsov, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. git-am: error: Failed to merge in the changes. hint: Use 'git am --show-current-patch' to see the fai

[ovs-dev] [PATCH ovn v6 2/4] northd, utils: support for RouteTables in LRs

2021-10-05 Thread Vladislav Odintsov
This patch 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_R