[ovs-dev] [PATCH ovn v2] controller: add ovn-set-local-ip option

-by: Vladislav Odintsov --- controller/chassis.c| 12 +++ controller/encaps.c | 37 + controller/ovn-controller.8.xml | 7 +++ ovn-sb.xml | 9 tests/ovn-controller.at | 9 5 files changed, 61

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

Hi Dumitru, I’ve got a general question about what the logical topology you’d like to implement. Also, please see inline comments. Regards, Vladislav Odintsov > On 14 Feb 2022, at 15:51, Dumitru Ceara wrote: > > Hi Vladislav, > > I was looking at what options we have in O

Re: [ovs-dev] [PATCH ovn] vtep: provide option to check ovn-controller-vtep and ovn-northd versions

Thanks Numan, done: https://patchwork.ozlabs.org/project/ovn/patch/20220214175335.1956758-1-odiv...@gmail.com/ Regards, Vladislav Odintsov > On 11 Feb 2022, at 23:56, Numan Siddique wrote: > > On Wed, Feb 2, 2022 at 3:39 PM Vladislav Odintsov <mailto:odiv...@gmail.com>> wr

[ovs-dev] [PATCH ovn] NEWS: add info about ovn-controller-vtep version pinning

Signed-off-by: Vladislav Odintsov --- NEWS | 3 +++ 1 file changed, 3 insertions(+) diff --git a/NEWS b/NEWS index bf052838d..984e9977e 100644 --- a/NEWS +++ b/NEWS @@ -7,6 +7,9 @@ Post v21.12.0 - ovn-nbctl copp-list - ovn-nbctl ls-copp-add - ovn-nbctl lr-copp-add

Re: [ovs-dev] [PATCH ovn] controller: add ovn-set-local-ip option

Hi Mohammad, thanks for the review! I’ve added the requested testcase and submitted the v2: https://patchwork.ozlabs.org/project/ovn/patch/20220209233314.51948-1-odiv...@gmail.com/ Regards, Vladislav Odintsov > On 6 Feb 2022, at 12:59, Mohammad Heib wrote: > > Hi Vladislav, > &

[ovs-dev] [PATCH ovn v2] controller: add ovn-set-local-ip option

-by: Vladislav Odintsov --- controller/encaps.c | 37 + controller/ovn-controller.8.xml | 7 +++ tests/ovn-controller.at | 9 3 files changed, 40 insertions(+), 13 deletions(-) diff --git a/controller/encaps.c b/controller/encaps.c index

Re: [ovs-dev] Datapath downtime during OVN upgrade

are assigned again with the same IDs. Lport zones are not touched but for some reason traffic between them (lport-to-lport) is affected on ovn-controller start. Regards, Vladislav Odintsov > On 3 Feb 2022, at 19:56, Vladislav Odintsov wrote: > > UPD: > > I’ve checked ovn-cont

Re: [ovs-dev] Datapath downtime during OVN upgrade

atapaths here [1] So, this triggers zones removal and on next iteration the required zones are created back. What do you think? 0: https://github.com/ovn-org/ovn/blob/v21.12.0/controller/ovn-controller.c#L651 1: https://github.com/ovn-org/ovn/blob/v21.12.0/controller/ovn-controller.c#L658 Regard

Re: [ovs-dev] Datapath downtime during OVN upgrade

rack zones and modifies OFs even if nothing has changed? I mean, a "soft-restart" condition (when ovn-controller was stopped through unixctl socket with "--restart" option. Should I start ovn-controller somehow special? When I trigger "soft-restart", I expect t

[ovs-dev] [PATCH ovn] vtep: provide option to check ovn-controller-vtep and ovn-northd versions

database in the Global table's other_config:ovn-match-northd-version column/key. True value enforces check, while false or not defined values skip the check. The appropriate testcase is added as well. Signed-off-by: Vladislav Odintsov --- controller-vtep/ovn-controller-vtep.8.xml | 23

[ovs-dev] Datapath downtime during OVN upgrade

e having in memory lflows which are not consistent at that moment? 4. What does `br-int<->unix#272675: connection dropped (Broken pipe)` error means? What type of connection is broken? Is it normal? 5. Do you see anything else abnormal in the logs? Let me know if I have to give more inpu

Re: [ovs-dev] [PATCH ovn] vtep: set is-vtep to chassis's other_config if absent

Hi Dumitru, thanks for the review. I’ve addressed requested changes in v2: https://patchwork.ozlabs.org/project/ovn/patch/20220131173457.64391-1-odiv...@gmail.com/ Regards, Vladislav Odintsov > On 28 Jan 2022, at 20:06, Dumitru Ceara wrote: > > On 1/28/22 18:04, Dumitru Ceara wrote

[ovs-dev] [PATCH ovn v2] vtep: set is-vtep to chassis's other_config if absent

-vtep) tunnels") Signed-off-by: Vladislav Odintsov Acked-by: Dumitru Ceara --- controller-vtep/gateway.c| 5 + tests/ovn-controller-vtep.at | 8 2 files changed, 13 insertions(+) diff --git a/controller-vtep/gateway.c b/controller-vtep/gateway.c index 288772dc4..9fbfc0

[ovs-dev] [PATCH ovn] controller: add ovn-set-local-ip option

-by: Vladislav Odintsov --- controller/encaps.c | 37 + controller/ovn-controller.8.xml | 7 +++ 2 files changed, 31 insertions(+), 13 deletions(-) diff --git a/controller/encaps.c b/controller/encaps.c index 66e0cd8cd..3b0c92931 100644 --- a/controller

[ovs-dev] [PATCH ovn] vtep: set is-vtep to chassis's other_config if absent

) Signed-off-by: Vladislav Odintsov --- After this patch is accepted to main branch, it should be considered to be backported to branch-21.12, as the problem is shown originally in that version. --- controller-vtep/gateway.c | 5 + 1 file changed, 5 insertions(+) diff --git a/controller-vtep

Re: [ovs-dev] [PATCH ovn 3/3] northd: support mix of stateless ACL with lower priority than stateful

Thanks Dumitru. Regards, Vladislav Odintsov > On 10 Dec 2021, at 14:13, Dumitru Ceara wrote: > > On 12/9/21 17:12, Vladislav Odintsov wrote: >> Hi Dumitru, >> > > Hi Vladislav, > >> thanks for review & testing and pointing for the broken behaviou

Re: [ovs-dev] [PATCH] compat: handle NF_REPEAT error on nf_conntrack_in

Thanks Ilya! Regards, Vladislav Odintsov > On 9 Dec 2021, at 20:05, Ilya Maximets wrote: > > On 12/6/21 18:46, Gregory Rose wrote: >> >> >> On 11/26/2021 12:59 PM, Vladislav Odintsov wrote: >>> In patch [1] rpl_nf_conntrack_in was backported as static i

Re: [ovs-dev] [PATCH ovn 3/3] northd: support mix of stateless ACL with lower priority than stateful

how we can solve it. Anyway, I’ve managed to resolve my personal task, which initially was the reason for this patch, and if it's not needed for the community, I’m okay to close it for now. If somebody is interested in it, please let me know. What do you think? Regards, Vladislav Odintsov &g

Re: [ovs-dev] [PATCH ovn 0/3] Support mixing stateless and stateful ACLs regardless of their priority

Thanks Numan, I’ll wait for your comments on #3. Regards, Vladislav Odintsov > On 8 Dec 2021, at 19:13, Numan Siddique wrote: > > On Tue, Dec 7, 2021 at 11:43 AM Vladislav Odintsov wrote: >> >> Talking about patches 1 and 2 - they've got totally no negative impact, &g

Re: [ovs-dev] [PATCH ovn 0/3] Support mixing stateless and stateful ACLs regardless of their priority

be treated. Let me know your thoughts. regards, Vladislav Odintsov On 07.12.2021 19:14, Numan Siddique wrote: On Tue, Dec 7, 2021 at 3:58 AM Vladislav Odintsov wrote: On 01.12.2021 15:56, Vladislav Odintsov wrote: Currently if user has a stateless and statetul ACLs (allow-stateless and allow

Re: [ovs-dev] [PATCH ovn 0/3] Support mixing stateless and stateful ACLs regardless of their priority

On 01.12.2021 15:56, Vladislav Odintsov wrote: Currently if user has a stateless and statetul ACLs (allow-stateless and allow-related) in one port group or in one logical switch simultaneously, the stateless rules whould take precedence. This patch series adds support for mixing all the ACLs

[ovs-dev] [ovn] Cross port_group inport/outport ACL match bug?

add it to port-group2, this error doesn’t appear. It looks to me like the l_ctx_in->port_groups in consider_logical_flow__() is filled with only port-groups from current datapath logical switch port... OVN main branch codebase. Regards, Vladislav Odintsov _

Re: [ovs-dev] [PATCH ovn 1/3] Revert "northd: support HW VTEP with stateful datapath"

Signed-off-by: Vladislav Odintsov Regards, Vladislav Odintsov > On 1 Dec 2021, at 16:00, 0-day Robot wrote: > > Bleep bloop. Greetings Vladislav Odintsov, I am a robot and I have tried out > your patch. > Thanks for your contribution. > > I encountered some error th

[ovs-dev] [PATCH ovn 3/3] northd: support mix of stateless ACL with lower priority than stateful

should be okay. Signed-off-by: Vladislav Odintsov --- northd/northd.c | 92 + northd/ovn-northd.8.xml | 4 +- tests/ovn-northd.at | 48 - 3 files changed, 87 insertions(+), 57 deletions(-) diff --git a/northd/northd.c b/northd

[ovs-dev] [PATCH ovn 1/3] Revert "northd: support HW VTEP with stateful datapath"

This reverts commit 62ca8b9620cc1168ace6905575b7d36438363aed. --- northd/northd.c | 14 -- northd/ovn-northd.8.xml | 28 northd/ovn_northd.dl| 33 ++--- tests/ovn-northd.at | 2 -- 4 files changed, 2

[ovs-dev] [PATCH ovn 2/3] northd: send ingress packets from HW VTEP directly to L2_LKUP table

1: https://github.com/ovn-org/ovn/commit/62ca8b9620cc1168ace6905575b7d36438363aed Signed-off-by: Vladislav Odintsov --- northd/northd.c | 9 - northd/ovn-northd.8.xml | 9 + northd/ovn_northd.dl| 16 +--- 3 files changed, 30 insertions(+), 4 deletions(-)

[ovs-dev] [PATCH ovn 0/3] Support mixing stateless and stateful ACLs regardless of their priority

. This change requires next: Also, as an optimisation, traffic from HW VTEP switch in ingress datapath is passed from ls_in_l2_sec directly to ls_in_l2_lkup, as it doesn't need any processing in ingress pipeline except determining outport in ls_in_l2_lkup table. Vladislav Odintsov (3): Revert "n

Re: [ovs-dev] [PATCH ovn 1/2] northd: allow having VXLAN encaps in ovn-ic enabled setup

Thanks! regards, Vladislav Odintsov > On 29 Nov 2021, at 21:18, Numan Siddique wrote: > > On Tue, Nov 23, 2021 at 12:08 PM Vladislav Odintsov > wrote: >> >> In ovn-ic enabled setups, which also use VXLAN encapsulation >> (for instance, for VTEP-switches) i

Re: [ovs-dev] [ovn] problem: long tcp session instantiation with stateful ACLs

Hi Dumitru, Numan, I’ve sent a corresponding patch to openvswitch with my findings. It’d be great if you can take a look on it. Thanks. https://patchwork.ozlabs.org/project/openvswitch/patch/20211126205942.9354-1-odiv...@gmail.com/ Regards, Vladislav Odintsov > On 21 Sep 2021, at 14

[ovs-dev] [PATCH] compat: handle NF_REPEAT error on nf_conntrack_in

Signed-off-by: Vladislav Odintsov --- .../compat/include/net/netfilter/nf_conntrack_core.h | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/datapath/linux/compat/include/net/netfilter/nf_conntrack_core.h b/datapath/linux/compat/include/net/netfilter

[ovs-dev] [PATCH ovn] tests: fix flaky VXLAN check port/datapath key space limits test

-21.09.90/tests/testsuite.dir/at-groups/104/stdout 2021-11-23 17:34:51.644406189 +0300 $ @@ -1,2 +1,2 @@ $ -1 $ +5000 This patch fixes such fail by adding await for chassis to be registered in SB DB. Signed-off-by: Vladislav Odintsov --- tests/ovn.at | 1 + 1 file changed, 1 insertion

[ovs-dev] [PATCH ovn 2/2] tests: fix flaky VXLAN check port/datapath key space limits test

/tests/testsuite.dir/at-groups/104/stdout 2021-11-23 17:34:51.644406189 +0300 @@ -1,2 +1,2 @@ -1 +5000 This patch fixes such fail by adding await for chassis to be registered in SB DB. Signed-off-by: Vladislav Odintsov --- tests/ovn.at | 1 + 1 file changed, 1 insertion(+) diff --git a/tests

[ovs-dev] [PATCH ovn 1/2] northd: allow having VXLAN encaps in ovn-ic enabled setup

MIN_MULTICAST. This patch allows having such setup. Fixes: fd44d7595 (Enforce datapath and port key constraints in vxlan mode) Signed-off-by: Vladislav Odintsov --- northd/northd.c | 4 +++- tests/ovn-ic.at | 3 +++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/northd/northd.c b/no

Re: [ovs-dev] [PATCH] ofproto: Fix resource usage explosion due to removal of large number of flows.

Thanks for the patch! Tested-by: Vladislav Odintsov Regards, Vladislav Odintsov > On 22 Nov 2021, at 18:23, Ilya Maximets wrote: > > While removing flows, removal itself is deferred, so classifier changes > performed already from the RCU thread. This way every deferred remova

Re: [ovs-dev] [PATCH] ofproto: Fix resource usage explosion while processing bundled FLOW_MOD.

I’ve re-checked and it really stopped growing after a couple of iterations. Thanks again for the patches. Regards, Vladislav Odintsov > On 23 Nov 2021, at 02:03, Ilya Maximets wrote: > > On 11/22/21 20:54, Vladislav Odintsov wrote: >> Hi Ilya, >> >> I’ve teste

Re: [ovs-dev] [PATCH] ofproto: Fix resource usage explosion while processing bundled FLOW_MOD.

sp4 170.9 MiB + 1.4 MiB = 172.2 MiB ovs-vswitchd 170.9 MiB + 1.4 MiB = 172.2 MiB ovs-vswitchd 170.9 MiB + 1.4 MiB = 172.2 MiB ovs-vswitchd Can this be also related to above problem? Regards, Vladislav Odintsov > On 22 Nov 2021, at 18:25, Ilya Maximets wrote: > > On

Re: [ovs-dev] [PATCH] ofproto: Fix resource usage explosion while processing bundled FLOW_MOD.

mory and then got killed by OOM-killer. Let me know if you need any additional info. Regards, Vladislav Odintsov > On 22 Nov 2021, at 11:07, Vladislav Odintsov wrote: > > Thanks for the patch Ilya! > > I’ve tested this with my setup in OVN similar to described in message by link > a

Re: [ovs-dev] [PATCH] ofproto: Fix resource usage explosion while processing bundled FLOW_MOD.

load about 2-4%. I’ve tested this with both: OVS v2.13.5 and master branch. Tested-by: Vladislav Odintsov Regards, Vladislav Odintsov > On 20 Nov 2021, at 02:07, Ilya Maximets wrote: > > While processing a bundle, OVS will add all new and modified rules > to classifiers. Cla

[ovs-dev] ovs-vswitchd too large memory consumption with OVN stateless ACL

us problem for OVN transport nodes. Thanks. Regards, Vladislav Odintsov ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH ovn v8 2/5] northd: make connected routes have higher priority than static

Done: https://patchwork.ozlabs.org/project/ovn/patch/2029160721.60612-2-odiv...@gmail.com/ Regards, Vladislav Odintsov > On 19 Nov 2021, at 17:55, Numan Siddique wrote: > > On Fri, Nov 19, 2021 at 8:12 AM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrote: >>

[ovs-dev] [PATCH ovn v9 5/6] ic: add support for routing tables in adv/learn routes

h port and configured as ECMP routes. Note 2: static routes within route tables will be advertised and learned only if interconnecting transit switch's LRPs have same value in options:route_table as NB:Logical_Router_Static_Route or ICSB:Route route_table column value. Signed-off-by: Vladislav Odintsov Re

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

TING flows for routes which have non-empty `route_table` field are prefixed in match portion with "reg7 == " where id is an uniqly-generated route_table id (if any) or zero if route has empty route_table value. Signed-off-by: Vladislav Odintsov Acked-by: Numan Siddique --- lib/ovn-u

[ovs-dev] [PATCH ovn v9 6/6] ic: don't learn routes which have local GW

with already installed non-learned NB route, such route wouldn't be learned. Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 30 -- tests/ovn-ic.at | 49 +++ utilities/ovn-nbctl.c | 4 +++- 3 files changed, 80

[ovs-dev] [PATCH ovn v9 3/6] northd: make connected routes have higher priority than static

multiplied by 3 + route origin offset (0 - source-based route; 1 - static route; 2 - directly-connected route2). Also, enlarge prio for generic records in lr_in_ip_routing stage by 1. Signed-off-by: Vladislav Odintsov --- northd/northd.c | 50

[ovs-dev] [PATCH ovn v9 2/6] ic: maintain route origin - connected/static

routing stage lflows. Signed-off-by: Vladislav Odintsov Acked-by: Numan Siddique --- ic/ovn-ic.c | 34 +++ lib/ovn-util.h | 3 +++ ovn-ic-sb.ovsschema | 7 -- ovn-ic-sb.xml | 10 ovn-nb.xml | 14 +++ test

[ovs-dev] [PATCH ovn v9 1/6] northd: fix symmetric ECMP reply flows documentation

This patch fixes a flow priority which is added in IP Defrag table and adds priority value for symmetric ECMP reply flow in IP Routing table. Fixes: 384a7c6 (northd: Refactor Logical Flows for routers with DNAT/Load Balancers) Signed-off-by: Vladislav Odintsov --- northd/ovn-northd.8.xml | 18

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

m 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 Ro

Re: [ovs-dev] [PATCH ovn v8 2/5] northd: make connected routes have higher priority than static

/patch/2027203545.46142-1-odiv...@gmail.com/ Regards, Vladislav Odintsov > On 18 Nov 2021, at 19:50, Numan Siddique wrote: > > Hi Vladislav, > > Once v9 is ready, please submit them. I'll take a look. > > Thanks > Numan > > > On Thu, Nov 18

Re: [ovs-dev] [PATCH ovn v8 2/5] northd: make connected routes have higher priority than static

Thanks, Han. Please see inline. Regards, Vladislav Odintsov > On 18 Nov 2021, at 10:26, Han Zhou wrote: > > On Wed, Nov 17, 2021 at 10:10 PM Vladislav Odintsov <mailto:odiv...@gmail.com>> > wrote: >> >> Great, thanks. >> >> Hi @Han, >> >

Re: [ovs-dev] [PATCH ovn v8 2/5] northd: make connected routes have higher priority than static

Great, thanks. Hi @Han, I’d like you to look at the patch series too. Would you have time on it? If yes, could you redirect me on terms please. Thanks. Regards, Vladislav Odintsov > On 18 Nov 2021, at 00:05, Numan Siddique wrote: > > On Wed, Nov 17, 2021 at 3:38 PM Vladislav

Re: [ovs-dev] [PATCH ovn v8 2/5] northd: make connected routes have higher priority than static

I’ve submitted a patch [1] with my findings. Also, if no comments for other my patches from this patch series, I can submit a new version. Should I? 1: https://patchwork.ozlabs.org/project/ovn/patch/2027203545.46142-1-odiv...@gmail.com/ Regards, Vladislav Odintsov > On 17 Nov 2021, at

[ovs-dev] [PATCH ovn] northd: fix symmetric ECMP reply flows documentation

This patch fixes a flow priority which is added in IP Defrag table and adds priority value for symmetric ECMP reply flow in IP Routing table. Fixes: 384a7c6 (northd: Refactor Logical Flows for routers with DNAT/Load Balancers) Signed-off-by: Vladislav Odintsov --- northd/ovn-northd.8.xml | 18

Re: [ovs-dev] [PATCH ovn v8 2/5] northd: make connected routes have higher priority than static

for other flow changes was added, but I’ve committed it to a wrong patch (#3). I’ll move documentation update between patches. Regards, Vladislav Odintsov > On 17 Nov 2021, at 13:51, Vladislav Odintsov wrote: > > Hi Numan, > > Thanks for the review. > Sure I will fix t

Re: [ovs-dev] [PATCH ovn v8 1/5] ic: maintain route origin - connected/static

but maybe I’m wrong here. Regards, Vladislav Odintsov > On 17 Nov 2021, at 05:13, Numan Siddique wrote: > > On Sat, Nov 13, 2021 at 4:44 AM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrote: >> >> This commits adds ability to save route's origin while I

Re: [ovs-dev] [PATCH ovn v8 2/5] northd: make connected routes have higher priority than static

Hi Numan, Thanks for the review. Sure I will fix this. Should I wait for more comments or that’s all and I can send v9? Regards, Vladislav Odintsov > On 17 Nov 2021, at 05:17, Numan Siddique wrote: > > On Sat, Nov 13, 2021 at 4:44 AM Vladislav Odintsov <mailto:odiv...@gmail

[ovs-dev] [PATCH ovn v8 3/5] northd, utils: support for RouteTables in LRs

TING flows for routes which have non-empty `route_table` field are prefixed in match portion with "reg7 == " where id is an uniqly-generated route_table id (if any) or zero if route has empty route_table value. Signed-off-by: Vladislav Odintsov Acked-by: Numan Siddique --- northd/no

[ovs-dev] [PATCH ovn v8 4/5] ic: add support for routing tables in adv/learn routes

h port and configured as ECMP routes. Note 2: static routes within route tables will be advertised and learned only if interconnecting transit switch's LRPs have same value in options:route_table as NB:Logical_Router_Static_Route or ICSB:Route route_table column value. Signed-off-by: Vladislav Odintsov Re

[ovs-dev] [PATCH ovn v8 5/5] ic: don't learn routes which have local GW

with already installed non-learned NB route, such route wouldn't be learned. Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 30 -- tests/ovn-ic.at | 49 +++ utilities/ovn-nbctl.c | 4 +++- 3 files changed, 80

[ovs-dev] [PATCH ovn v8 2/5] northd: make connected routes have higher priority than static

multiplied by 3 + route origin offset (0 - source-based route; 1 - directly- connected route; 2 - static route). Also, enlarge prio for generic records in lr_in_ip_routing stage by 1. Signed-off-by: Vladislav Odintsov --- northd/northd.c | 50

[ovs-dev] [PATCH ovn v8 1/5] ic: maintain route origin - connected/static

_in_ip_routing stage lflows. Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 34 +++ lib/ovn-util.h | 3 +++ ovn-ic-sb.ovsschema | 7 -- ovn-ic-sb.xml | 10 tests/ovn-ic.at | 57 + 5 files ch

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

rk.ozlabs.org/project/ovn/patch/20210830195707.98529-1-odiv...@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

Re: [ovs-dev] problem: long tcp session instantiation with conntrack and OOT ovs kmod

Hi Greg, thanks for the response. I’ll be grateful if you can find a time on this. Thanks. Regards, Vladislav Odintsov > On 4 Nov 2021, at 00:48, Gregory Rose wrote: > > > > On 11/2/2021 9:50 AM, Vladislav Odintsov wrote: >> Hi, >> it’s just a quick reminder, th

[ovs-dev] [PATCH ovn v7 3/5] northd, utils: support for RouteTables in LRs

with appropriate route_table. Signed-off-by: Vladislav Odintsov Acked-by: Numan Siddique --- northd/northd.c | 155 +--- northd/ovn-northd.8.xml | 63 +++-- ovn-nb.ovsschema| 5 +- ovn-nb.xml | 30 +++ tests/ovn-ic.at | 4 + tests/ovn-nbctl.at

[ovs-dev] [PATCH ovn v7 4/5] ic: add support for routing tables in adv/learn routes

ransit switch port and configured as ECMP routes. Note 2: static routes within route tables will be advertised and learned only if interconnecting transit switch's LRPs will have options:route_table same value as route's route_table value. Signed-off-by: Vladislav Odintsov Reviewed-by: Numan Si

[ovs-dev] [PATCH ovn v7 5/5] ic: don't learn routes which have local GW

with already installed non-learned NB route, such route wouldn't be learned. Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 30 -- tests/ovn-ic.at | 49 +++ utilities/ovn-nbctl.c | 4 +++- 3 files changed, 80

[ovs-dev] [PATCH ovn v7 2/5] northd: make connected routes have higher priority than static

by 'slots' count (currently 3) plus route origin offset (0 - source-based route; 1 - directly- connected route; 2 - static route). Also, enlarge prio for generic records in lr_in_ip_routing stage by 1. Signed-off-by: Vladislav Odintsov --- northd/northd.c | 42

[ovs-dev] [PATCH ovn v7 1/5] ic: maintain route origin - connected/static

_in_ip_routing stage lflows. Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 34 +++ lib/ovn-util.h | 3 +++ ovn-ic-sb.ovsschema | 7 -- ovn-ic-sb.xml | 5 tests/ovn-ic.at | 57 + 5 files ch

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

eparately, 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 (5): ic: maintain route origin - connected/static northd: make connected rout

Re: [ovs-dev] [ovn] chassis with VXLAN breaks setup which is using OVN-IC

That sounds okay. But the problem is that if one have at least one chassis, which even doesn’t act in OVN IC, it will break ovn interconnected switches and LRs. Regards, Vladislav Odintsov > On 9 Nov 2021, at 18:46, Numan Siddique wrote: > > On Mon, Nov 8, 2021 at 5:05 PM Numan

[ovs-dev] [ovn] chassis with VXLAN breaks setup which is using OVN-IC

can be supported, but maybe I’m missing something. Can somebody help on this please? Thanks. 1: https://github.com/ovn-org/ovn/commit/fd44d75959cedcedf1f103173be1d9fa1abd9cb8 Regards, Vladislav Odintsov ___ dev mailing list d...@openvswitch.org ht

Re: [ovs-dev] problem: long tcp session instantiation with conntrack and OOT ovs kmod

Hi, it’s just a quick reminder, that if you know how to debug/fix this, please let me know. Thanks. Regards, Vladislav Odintsov > On 11 Oct 2021, at 15:48, Vladislav Odintsov wrote: > > Hi Greg, Pravin, > > I was suggested to contact you with my OVS/conntrack/OOT KMOD proble

Re: [ovs-dev] [ovn] bug: ovn-northd was blocked for changes handling

value (for instance, 2) and then ovn-northd tries to update tunnel_key and fails for some reason. Couldn’t it be a scenario for the duplicated datapath’s in sb db? Regards, Vladislav Odintsov > On 26 Oct 2021, at 03:55, Han Zhou wrote: > > On Mon, Oct 4, 2021 at 3:02 PM Vladislav Odints

Re: [ovs-dev] [ovn] bug: ovn-northd was blocked for changes handling

Hi Han, could you take a look on this, please? Regards, Vladislav Odintsov > On 5 Oct 2021, at 01:02, Vladislav Odintsov wrote: > > Hi, > > I’ve faced with a next issue using latest OVN (master branch) with OVN-IC > enabled: > ovn-northd CPU utilisation was at ~70

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

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 >> >>

Re: [ovs-dev] [PATCH ovn v6 1/4] ic: process only local port_bindings

Regards, Vladislav Odintsov > On 14 Oct 2021, at 06:23, Han Zhou wrote: > > > > On Tue, Oct 5, 2021 at 1:25 PM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrote: > > > > This commit adds a small optimization by utilizing ovsdb_index > >

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

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 <mailto:odiv...@gmail.com>> wrote: > > > > This patch extends Logical Router's routing fu

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

Hi Han, could you please take a look on this when you have time? Thanks. Regards, Vladislav Odintsov > On 5 Oct 2021, at 23:24, Vladislav Odintsov wrote: > > v5 -> v6: > - 2 memory leak bugs were fixed in ovn-ic code, which were introduced in >previous versions of #3 pa

[ovs-dev] problem: long tcp session instantiation with conntrack and OOT ovs kmod

t problem for my installation. Thanks in advance. 1: https://mail.openvswitch.org/pipermail/ovs-dev/2021-September/387623.html 2: https://docs.openvswitch.org/en/latest/tutorials/ovs-conntrack/ Regards, Vladislav Odintsov ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH ovn] tests: fix flaky "ovn-ic -- gateway sync" test

Thanks Numan. Regards, Vladislav Odintsov > On 8 Oct 2021, at 20:56, Numan Siddique wrote: > > On Fri, Oct 8, 2021 at 5:56 AM Mark Gray <mailto:mark.d.g...@redhat.com>> wrote: >> >> On 07/10/2021 19:04, Vladislav Odintsov wrote: >>> Sometimes when o

[ovs-dev] [PATCH ovn] tests: fix flaky "ovn-ic -- gateway sync" test

# -Encap geneve # -options: {csum="true"} # -options: {csum="true"} # -ip: "192.168.0.1" # -ip: "192.168.0.1" 1069. ovn-ic.at:119: 1069. ovn-ic -- gateway sync -- ovn-northd -- dp-groups=no (ovn-ic.at:119): FAILED (ovn-ic.at

[ovs-dev] [PATCH ovn] tests: fix flaky "ovn-ic -- gateway sync" test

;true"} -options: {csum="true"} -ip: "192.168.0.1" -ip: "192.168.0.1" 1069. ovn-ic.at:119: 1069. ovn-ic -- gateway sync -- ovn-northd -- dp-groups=no (ovn-ic.at:119): FAILED (ovn-ic.at:119) Signed-off-by: Vladislav Odintsov --- tests/ovn-ic.at | 1

[ovs-dev] [PATCH ovn v6 4/4] ic: don't learn routes which have local GW

with already installed non-learned NB route, such route wouldn't be learned. Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 30 -- tests/ovn-ic.at | 49 +++ utilities/ovn-nbctl.c | 4 +++- 3 files changed, 80

[ovs-dev] [PATCH ovn v6 3/4] ic: add support for routing tables in adv/learn routes

ransit switch port and configured as ECMP routes. Note 2: static routes within route tables will be advertised and learned only if interconnecting transit switch's LRPs will have options:route_table same value as route's route_table value. Signed-off-by: Vladislav Odintsov Reviewed-by: Numan Si

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

with appropriate route_table. Signed-off-by: Vladislav Odintsov Acked-by: Numan Siddique --- northd/northd.c | 159 --- northd/ovn-northd.8.xml | 63 -- ovn-nb.ovsschema| 5 +- ovn-nb.xml | 30 +++ tests/ovn-ic.at | 4 + tests/ovn-nbctl.at

[ovs-dev] [PATCH ovn v6 1/4] ic: process only local port_bindings

availability_zone. Signed-off-by: Vladislav Odintsov Acked-by: Numan Siddique --- ic/ovn-ic.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/ic/ovn-ic.c b/ic/ovn-ic.c index 99356253d..303e93a4f 100644 --- a/ic/ovn-ic.c +++ b/ic/ovn-ic.c @@ -68,6 +68,7 @@ struct

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

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 (4): ic: process only local port_bindings northd

[ovs-dev] [PATCH ovn v5 3/4] ic: add support for routing tables in adv/learn routes

ransit switch port and configured as ECMP routes. Note 2: static routes within route tables will be advertised and learned only if interconnecting transit switch's LRPs will have options:route_table same value as route's route_table value. Signed-off-by: Vladislav Odintsov Reviewed-by: Numan Si

[ovs-dev] [PATCH ovn v5 5/4] ic: add support for routing tables in adv/learn routes

ransit switch port and configured as ECMP routes. Note 2: static routes within route tables will be advertised and learned only if interconnecting transit switch's LRPs will have options:route_table same value as route's route_table value. Signed-off-by: Vladislav Odintsov Reviewed-by: Numan Si

[ovs-dev] [PATCH ovn v5 3/4] ic: add support for routing tables in adv/learn routes

ransit switch port and configured as ECMP routes. Note 2: static routes within route tables will be advertised and learned only if interconnecting transit switch's LRPs will have options:route_table same value as route's route_table value. Signed-off-by: Vladislav Odintsov Reviewed-by: Numan Si

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

://patchwork.ozlabs.org/project/ovn/list/?series=265501 Regards, Vladislav Odintsov > On 4 Oct 2021, at 19:20, Numan Siddique wrote: > > On Sun, Sep 19, 2021 at 5:24 PM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrote: >> >> Signed-off-by: Vladislav Odintsov >>

[ovs-dev] [PATCH ovn v5 3/4] ic: add support for routing tables in adv/learn routes

ransit switch port and configured as ECMP routes. Note 2: static routes within route tables will be advertised and learned only if interconnecting transit switch's LRPs will have options:route_table same value as route's route_table value. Signed-off-by: Vladislav Odintsov Reviewed-by: Numan Si

[ovs-dev] [PATCH ovn v5 4/4] ic: don't learn routes which have local GW

with already installed non-learned NB route, such route wouldn't be learned. Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 30 -- tests/ovn-ic.at | 49 +++ utilities/ovn-nbctl.c | 4 +++- 3 files changed, 80

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

with appropriate route_table. Signed-off-by: Vladislav Odintsov Acked-by: Numan Siddique --- northd/northd.c | 159 --- northd/ovn-northd.8.xml | 63 -- ovn-nb.ovsschema| 5 +- ovn-nb.xml | 30 +++ tests/ovn-ic.at | 4 + tests/ovn-nbctl.at

[ovs-dev] [PATCH ovn v5 1/4] ic: process only local port_bindings

availability_zone. Signed-off-by: Vladislav Odintsov Acked-by: Numan Siddique --- ic/ovn-ic.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/ic/ovn-ic.c b/ic/ovn-ic.c index 99356253d..303e93a4f 100644 --- a/ic/ovn-ic.c +++ b/ic/ovn-ic.c @@ -68,6 +68,7 @@ struct

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

authors list. Vladislav Odintsov (4): ic: process only local port_bindings 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

[ovs-dev] [ovn] bug: ovn-northd was blocked for changes handling

here is that northd was entirely blocked. No processing to SB was done. Even ovn-controllers couldn’t claim new ports. No changes were propagated, so this is critial issue. Can anybody help understand the reason for this? Regards, Vladislav Odintsov ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [ovn] 21.09.0 release not ready yet?

be in this release: https://patchwork.ozlabs.org/project/ovn/patch/20210916173028.9527-1-odiv...@gmail.com/ Thanks. Regards, Vladislav Odintsov ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [PATCH ovn] rhel: replace try-restart with restart in ovn-controller %postun

fixes such situation by checking actual ovn-controller service status and doing an unconditional restart if service was running before upgrade. [1] https://github.com/ovn-org/ovn/commit/8540c544f0e67d3dc475bbeb350ea3053a1772dd Signed-off-by: Vladislav Odintsov --- rhel/ovn-fedora.spec.in | 17

Re: [ovs-dev] [ovn] problem: long tcp session instantiation with stateful ACLs

Hi Dumitru, are you talking about any specific _mising_ patch? Regards, Vladislav Odintsov > On 16 Sep 2021, at 19:09, Dumitru Ceara wrote: > > On 9/16/21 4:18 PM, Vladislav Odintsov wrote: >> Sorry, by OOT I meant non-inbox kmod. >> I’ve tried to use inbox kernel module

[ovs-dev] [PATCH ovn v4 3/4] ic: add support for routing tables in adv/learn routes

ransit switch port and configured as ECMP routes. Note 2: static routes within route tables will be advertised and learned only if interconnecting transit switch's LRPs will have options:route_table same value as route's route_table value. Signed-off-by: Vladislav Odintsov --- NEWS

<    1   2   3   4   5   >