Re: [ovs-dev] [PATCH ovn branch-23.03] ovs: Bump submodule to tip of OVS branch-3.2.

2024-10-11 Thread Odintsov Vladislav
On 11.10.2024 22:59, Dumitru Ceara wrote: Sorry for the confusion. I was trying to suggest to bump the submodule on all OVN stable branches to the current tip of a stable OVS release branch. That's what I think our documentation suggests: https://github.com/ovn-org/ovn/blob/main/Documentation

Re: [ovs-dev] [PATCH ovn branch-23.03] ovs: Bump submodule to tip of OVS branch-3.2.

2024-10-11 Thread Odintsov Vladislav
On 11.10.2024 14:00, Dumitru Ceara wrote: On 10/7/24 12:45, Vladislav Odintsov wrote: From: Dumitru Ceara This picks up the following relevant commit: cd8ffc956c3c ovs-atomic: Fix inclusion of Clang header by GCC 14. Without this builds on Fedora 40 (rawhide) are b

Re: [ovs-dev] [PATCH ovn branch-23.03] ovs: Bump submodule to tip of OVS branch-3.2.

2024-10-11 Thread Odintsov Vladislav
Hi Dumitru, On 11.10.2024 14:00, Dumitru Ceara wrote: On 10/7/24 12:45, Vladislav Odintsov wrote: From: Dumitru Ceara This picks up the following relevant commit: cd8ffc956c3c ovs-atomic: Fix inclusion of Clang header by GCC 14. Without this builds on Fedora 40 (r

Re: [ovs-dev] [ovn] additional_chassis LSP doesn't trigger BFD sessions to come up

2024-09-19 Thread Odintsov Vladislav
Hi Numan, thanks for a quick reply! On 19.09.2024 16:28, Numan Siddique wrote: On Thu, Sep 19, 2024 at 8:18 AM Odintsov Vladislav <mailto:VlOdintsov@k2.cloud> wrote: Forgot to mention. ovn-controller logs when I call "finalize" after wait of 15 seconds. Here the VM port

Re: [ovs-dev] [ovn] additional_chassis LSP doesn't trigger BFD sessions to come up

2024-09-19 Thread Odintsov Vladislav
interface ovn-vad08-0 2024-09-19T12:15:45.913Z|00115|ovn_bfd|INFO|Enabled BFD on interface ovn-vad09-0 2024-09-19T12:15:45.913Z|00116|ovn_bfd|INFO|Enabled BFD on interface ovn-vad08-0 Also, wanted to note that these tests are run on the ovn-22.09.4 (branch-22.09) code. On 19.09.2024 14:13, Odint

[ovs-dev] [ovn] additional_chassis LSP doesn't trigger BFD sessions to come up

2024-09-19 Thread Odintsov Vladislav
Hi, I'm trying to utilize requested-chassis feature with two chassis in a list with activation-strategy=rarp for VM migration and found that BFD sessions (to configured ha-chassis-group) got configured only after port-binding of a migrating VM in fully claimed as a "main" chassis to the destin

Re: [ovs-dev] [RFC ovn-heater] OVN scale testing with OpenStack workloads - questionnaire

2023-07-12 Thread Odintsov Vladislav
On 12 Jul 2023, at 23:09, Dumitru Ceara wrote: On 7/12/23 21:46, Odintsov Vladislav wrote: Hi Dumitru, Hi Vladislav, first of all, I’l like to mention that’s a great idea! I’m wondering wether ovn interconnection infrastructure and ovn vtep could also be a part of this testing? There&#

Re: [ovs-dev] [RFC ovn-heater] OVN scale testing with OpenStack workloads - questionnaire

2023-07-12 Thread Odintsov Vladislav
Hi Dumitru, first of all, I’l like to mention that’s a great idea! I’m wondering wether ovn interconnection infrastructure and ovn vtep could also be a part of this testing? On 12 Jul 2023, at 22:38, Dumitru Ceara wrote: On 7/12/23 14:42, Frode Nordahl wrote: On Wed, Jul 12, 2023 at 2:31 PM Fe

Re: [ovs-dev] [PATCH ovn] ovn-ic: Only monitor useful tables and columns.

2023-01-11 Thread Odintsov Vladislav
Thank you too, Dumitru! Regards, Vladislav Odintsov On 11 Jan 2023, at 18:01, Dumitru Ceara mailto:dce...@redhat.com>> wrote: On 1/11/23 12:36, Dumitru Ceara wrote: On 1/10/23 23:21, Odintsov Vladislav wrote: I’ve posted a bunch of series/single patches with the backport. Thanks a l

Re: [ovs-dev] [PATCH ovn] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Odintsov Vladislav
/ovn-org/ovn/commit/18f719968 Regards, Vladislav Odintsov On 10 Jan 2023, at 22:08, Odintsov Vladislav mailto:vlodint...@croc.ru>> wrote: I’ve checked and I’ll try to post backport patches to branches 22.06 and older. Some of them have briken ci, I’ll incorporate ci fixes in appropriate patch

Re: [ovs-dev] [PATCH ovn] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Odintsov Vladislav
, Vladislav Odintsov On 10 Jan 2023, at 14:55, Dumitru Ceara mailto:dce...@redhat.com>> wrote: On 1/10/23 10:51, Odintsov Vladislav wrote: Hi Mark, Dumitru, Hi Vladislav, thanks for this change! Would you consider backporting this patch down to branches? It seems very useful. I'm O

Re: [ovs-dev] [PATCH ovn] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Odintsov Vladislav
Hi Mark, Dumitru, thanks for this change! Would you consider backporting this patch down to branches? It seems very useful. Regards, Vladislav Odintsov On 9 Jan 2023, at 22:48, Mark Michelson mailto:mmich...@redhat.com>> wrote: Thanks Dumitru, Acked-by: Mark Michelson mailto:mmich...@redhat.

Re: [ovs-dev] [PATCH ovn] northd: support vtep LSP-attached LS to use L3 services

2022-08-15 Thread Odintsov Vladislav
Thanks Numan. regards, Vladislav Odintsov > On 16 Aug 2022, at 04:56, Numan Siddique wrote: > > On Tue, Aug 16, 2022 at 12:19 AM Vladislav Odintsov > wrote: >> >> If LRP's logical switch has attached LSP of vtep type, the >> is_chassis_resident() part is not added to lflow to allow traffic

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

2022-02-01 Thread Odintsov Vladislav
Thanks Mark. Regards, Vladislav Odintsov On 1 Feb 2022, at 19:23, Mark Michelson mailto:mmich...@redhat.com>> wrote: Thanks, folks. I pushed the changes to main and to branch-21.12. On 2/1/22 03:46, Odintsov Vladislav wrote: Thanks Dumitru. I agree, that chaning one line in a file d

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

2022-02-01 Thread Odintsov Vladislav
Thanks Dumitru. I agree, that chaning one line in a file doesn’t help much :) This work should be done at the whole file someday. Regards, Vladislav Odintsov On 1 Feb 2022, at 11:33, Dumitru Ceara mailto:dce...@redhat.com>> wrote: On 1/31/22 18:34, Vladislav Odintsov wrote: After commit [0] th

Re: [ovs-dev] [PATCH] datapath: handle DNAT tuple collision

2021-12-02 Thread Odintsov Vladislav
Thanks for the backport from upstream! May I request backport patch [1] and this patch down to supported branches? Thanks. 1: https://github.com/openvswitch/ovs/commit/6a101a6c8372570a30e0f8edb558c8a69cc80e7d Regards, Vladislav Odintsov On 12 Oct 2021, at 22:30, Ilya Maximets mailto:i.maxim..

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

2021-11-23 Thread Odintsov Vladislav
Reported-at: https://mail.openvswitch.org/pipermail/ovs-dev/2021-November/389256.html Regards, Vladislav Odintsov On 23 Nov 2021, at 20:37, Vladislav Odintsov mailto:odiv...@gmail.com>> wrote: OVN requested-tnl-key VXLAN checks affect tunnel_key in datapath_binding record only on datapath tunn

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

2021-11-23 Thread Odintsov Vladislav
Thanks Numan! regards, Vladislav Odintsov > On 23 Nov 2021, at 00:35, Numan Siddique wrote: > > On Fri, Nov 19, 2021 at 11:07 AM Vladislav Odintsov > wrote: >> >> v8 -> v9: >> - Fix documentation errors. >> - Fix commit messages. >> >> v7 -> v8: >> - Updated manpages with and commit mes

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

2021-11-17 Thread Odintsov Vladislav
+ Han Regards, Vladislav Odintsov On 18 Nov 2021, at 09:10, Vladislav Odintsov mailto:odiv...@gmail.com>> wrote: 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 Odintso

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

2021-11-13 Thread Odintsov Vladislav
mailto:num...@ovn.org>> wrote: On Thu, Nov 11, 2021 at 2:18 PM Odintsov Vladislav mailto:vlodint...@croc.ru>> 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

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

2021-11-11 Thread Odintsov Vladislav
;> wrote: > > > > On Mon, Oct 18, 2021 at 6:35 AM Odintsov Vladislav > > mailto:vlodint...@croc.ru>> > > wrote: > > > > > > > > > > > > regards, > > > Vladislav Odintsov > > > > > > > On 16 Oct 2021, at

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

2021-11-10 Thread Odintsov Vladislav
Hi Ihar, also it seems that the test, which was added in a mentioned commit, is flaky. I continuously see 2 or 3 runs of 10 are failed on my dev VM. With verbose logging fail looks like this: 103. ovn.at:3708: testing VXLAN check port/datapath key space limits -- ovn-northd

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

2021-10-05 Thread Odintsov Vladislav
I’m sorry for this spam — I was trying to replace one patch in existing patch series, because found and fixed a bug. I’ll resend v6. Regards, Vladislav Odintsov On 5 Oct 2021, at 21:43, Vladislav Odintsov mailto:odiv...@gmail.com>> wrote: Previously support for multiple routing tables was adde

Re: [ovs-dev] [PATCH ovn] controller: Fall back to full recompute of pflow_output for vtep lport changes.

2021-09-15 Thread Odintsov Vladislav
anges, necessary flows are not added as expected. This is because the function physical_handle_flows_for_lport() in physical.c does not add flows required for vtep logical ports. These flows are added by physical_run(). So fall back to full recompute of pflow_output engine node when vtep lports

Re: [ovs-dev] [ovn] bug: vtep port binding update doesn't trigger flows update

2021-09-07 Thread Odintsov Vladislav
Hi Numan, thanks for a quick fix. I’ll test it in a few days and come back with the results. Regards, Vladislav Odintsov On 7 Sep 2021, at 20:18, Numan Siddique mailto:num...@ovn.org>> wrote: On Mon, Sep 6, 2021 at 4:08 AM Odintsov Vladislav mailto:vlodint...@croc.ru>> wrot

Re: [ovs-dev] [ovn] bug: vtep port binding update doesn't trigger flows update

2021-09-06 Thread Odintsov Vladislav
information do you need to handle L-P properly for there ports? I can test a possible solution. Regards, Vladislav Odintsov On 4 Sep 2021, at 19:25, Numan Siddique mailto:num...@ovn.org>> wrote: On Fri, Sep 3, 2021 at 7:26 PM Odintsov Vladislav mailto:vlodint...@croc.ru>> wrote: Hi

[ovs-dev] [ovn] bug: vtep port binding update doesn't trigger flows update

2021-09-03 Thread Odintsov Vladislav
Hi, On master branch I found a regression that OF flows not get installed when a new port_binding type=vtep appeared in sbdb. I found there is a comment that ovn-controller doesn’t know, what changed: https://github.com/ovn-org/ovn/blob/922c45f/controller/binding.c#L2431-L2436 Actually, this flo

Re: [ovs-dev] [PATCH ovn 2/3] northd: support for RouteTables in LRs

2021-08-31 Thread Odintsov Vladislav
Regards, Vladislav Odintsov On 31 Aug 2021, at 16:51, Numan Siddique mailto:num...@ovn.org>> wrote: On Tue, Aug 31, 2021 at 9:35 AM Odintsov Vladislav mailto:vlodint...@croc.ru>> wrote: Regards, Vladislav Odintsov On 31 Aug 2021, at 15:36, Numan Siddique mailto:nu

Re: [ovs-dev] [PATCH ovn 2/3] northd: support for RouteTables in LRs

2021-08-31 Thread Odintsov Vladislav
Regards, Vladislav Odintsov On 31 Aug 2021, at 15:36, Numan Siddique mailto:num...@ovn.org>> wrote: On Tue, Aug 31, 2021 at 7:50 AM Odintsov Vladislav mailto:vlodint...@croc.ru>> wrote: One more addition for the ID usage: In ovn-ic code I gust copy route_table value from NB to

Re: [ovs-dev] [PATCH ovn 2/3] northd: support for RouteTables in LRs

2021-08-31 Thread Odintsov Vladislav
On Mon, Aug 30, 2021 at 11:48 PM Odintsov Vladislav mailto:vlodint...@croc.ru>> wrote: > > Hi Han, > > Using Router Policies for the purpose of routing makes impossible to use > filtering on the LR level (using Router Policies, not ACLs), because in that > case routing flow

Re: [ovs-dev] [PATCH ovn 2/3] northd: support for RouteTables in LRs

2021-08-30 Thread Odintsov Vladislav
Hi Han, Using Router Policies for the purpose of routing makes impossible to use filtering on the LR level (using Router Policies, not ACLs), because in that case routing flows and filter flows would be in the same lr stage. We offer to our customers ability to configure multiple routing tables,

Re: [ovs-dev] [PATCH ovn v4 5/5] controller: Improve ct zone handling.

2021-08-23 Thread Odintsov Vladislav
function for its peer. In this case lrp1. Please let me know your thoughts. Regards, Vladislav Odintsov On 23 Aug 2021, at 18:04, Numan Siddique mailto:num...@ovn.org>> wrote: On Sat, Aug 21, 2021 at 4:11 AM Odintsov Vladislav mailto:vlodint...@croc.ru>> wrote: Thanks Numan for the

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

2021-08-23 Thread 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 Ro

Re: [ovs-dev] [PATCH ovn v4 5/5] controller: Improve ct zone handling.

2021-08-21 Thread Odintsov Vladislav
On Fri, Aug 20, 2021 at 7:28 PM Numan Siddique mailto:num...@ovn.org>> wrote: On Fri, Aug 20, 2021 at 7:19 PM Odintsov Vladislav mailto:vlodint...@croc.ru>> wrote: Hi Numan, It looks like this patch introduces a regression. In my scenario with this commit my quite complicated virtua

Re: [ovs-dev] [PATCH ovn v4 5/5] controller: Improve ct zone handling.

2021-08-20 Thread Odintsov Vladislav
Hi Numan, It looks like this patch introduces a regression. In my scenario with this commit my quite complicated virtual topology is non-working and I have to call recompute to make things working. Debugging my problem I’ve found next problem: If we create LR and LRP, we can see that for this LR

Re: [ovs-dev] [PATCH ovn] controller-vtep: fix mmr and physical locators create/update

2021-05-28 Thread Odintsov Vladislav
I've posted a new version of this patch with requested changes: https://patchwork.ozlabs.org/project/ovn/patch/20210528082105.70086-1-odiv...@gmail.com/ Regards, Vladislav Odintsov On 27.05.2021, 23:00, "dev on behalf of 0-day Robot" wrote: Bleep bloop. Greetings Vladislav Odintsov, I

Re: [ovs-dev] [PATCH ovn] ovn-controller-vtep: Set chassis_name for newly created Encap.

2021-04-20 Thread Odintsov Vladislav
Mark Michelson On 4/20/21 6:32 AM, Dumitru Ceara wrote: > Without this ovn-controller-vtep fails to register new chassis when RBAC > is enabled. > > Reported-at: https://mail.openvswitch.org/pipermail/ovs-dev/2021-April/382207.html > Reported-by: O

Re: [ovs-dev] [ovn] transaction error in ovn-controller with configured RBAC on branch-20.06

2021-04-19 Thread Odintsov Vladislav
tch ovn-controller-vtep to another ovnsbdb port (without rbac engine), ovn-controller-vtep successfully adds chassis record, then I switch it back to rbac socket and continue working well. So, error occurs only on first run of chassis. When chassis exists in DB, things work well. Regards, Vlad

Re: [ovs-dev] [PATCH ovn 2/2] rhel: Add systemd-units for ovn-ic services

2021-02-03 Thread Odintsov Vladislav
Thanks! Regards, Vladislav Odintsov On 03.02.2021, 15:34, "dev on behalf of Numan Siddique" wrote: On Wed, Feb 3, 2021 at 2:15 PM wrote: > > From: Vladislav Odintsov > > New ovn-ic.service and ovn-ic-db.service systemd units were added > to manage ovn-ic and ovsd

Re: [ovs-dev] Reminder: OVN 21.03 soft freeze is 5 February

2021-02-02 Thread Odintsov Vladislav
Hi Mark, https://github.com/ovn-org/ovn/pull/65 This PR should be merged before new branch creation, as there are partial revert for rpm packaging changes. Currently it’s under Numan's review. Regards, Vladislav Odintsov On 01.02.2021, 22:52, "dev on behalf of Mark Michelson" wrote:

Re: [ovs-dev] [PATCH ovn v2 3/3] utilities: add conditional {start/stop}_ic_ovsdb on start_ic command

2021-01-25 Thread Odintsov Vladislav
I’ve updated PR https://github.com/ovn-org/ovn/pull/65 Could you please take a look on it? Regards, Vladislav Odintsov From: Han Zhou Date: Monday, 25 January 2021, 11:57 To: Odintsov Vladislav Cc: Han Zhou , Vladislav Odintsov , ovs dev Subject: Re: [ovs-dev] [PATCH ovn v2 3/3] utilities

Re: [ovs-dev] [PATCH ovn v2 3/3] utilities: add conditional {start/stop}_ic_ovsdb on start_ic command

2021-01-25 Thread Odintsov Vladislav
, he/she needs to have additional system-unit for OVN IC DB. So, the best option is – a new systemd-unit for OVN IC DBs. Right? Regards, Vladislav Odintsov From: Han Zhou Date: Monday, 25 January 2021, 10:59 To: Odintsov Vladislav Cc: Numan Siddique , Han Zhou , ovs dev , Vladislav Odintsov

Re: [ovs-dev] [PATCH ovn v2 3/3] utilities: add conditional {start/stop}_ic_ovsdb on start_ic command

2021-01-24 Thread Odintsov Vladislav
Hi Numan, Han, I understand you're busy on other tasks. Please take a look on this when you have time. Regards, Vladislav Odintsov On 18.01.2021, 11:51, "Odintsov Vladislav" wrote: Hi Numan, Han, A quick remind. Do you think such change is okay? Regards,

Re: [ovs-dev] [PATCH ovn v2 3/3] utilities: add conditional {start/stop}_ic_ovsdb on start_ic command

2021-01-18 Thread Odintsov Vladislav
Hi Numan, Han, A quick remind. Do you think such change is okay? Regards, Vladislav Odintsov On 12.01.2021, 15:49, "Odintsov Vladislav" wrote: Hi, Sorry for the long reply. There were long NY holidays in Russia. I do have same thoughts as Han. Maybe, it&#

Re: [ovs-dev] [PATCH ovn v2 3/3] utilities: add conditional {start/stop}_ic_ovsdb on start_ic command

2021-01-12 Thread Odintsov Vladislav
Hi, Sorry for the long reply. There were long NY holidays in Russia. I do have same thoughts as Han. Maybe, it's better to introduce separate option --ovn-manage-ic-ovsdb, which can have NO as default value and then just pass --ovn-manage-ic-ovsdb=yes in ovn-ic system-unit? In such situation cu

Re: [ovs-dev] [ovn] transaction error in ovn-controller with configured RBAC on branch-20.06

2020-12-09 Thread Odintsov Vladislav
Hi Dumitru, That’s good news, thanks for that! Regards, Vladislav Odintsov On 08.12.2020, 22:33, "Dumitru Ceara" wrote: On 12/8/20 8:28 PM, Dumitru Ceara wrote: > On 12/3/20 4:11 PM, Dumitru Ceara wrote: >> On 12/3/20 2:01 PM, Odintsov Vladislav wrote:

Re: [ovs-dev] [ovn] transaction error in ovn-controller with configured RBAC on branch-20.06

2020-12-03 Thread Odintsov Vladislav
rror: {"details":"RBAC rules for client \"04540082-b5b5-4ab5-9901-03ed445c772d\" role \"ovn-controller\" prohibit modification of table \"Encap\".","error":"permission error"} 2020-12-03T12:56:38.655Z|00016|main|INFO|OVNSB commit

[ovs-dev] [ovn] transaction error in ovn-controller with configured RBAC on branch-20.06

2020-12-03 Thread Odintsov Vladislav
Hi, It seems, that I see regression with claim port functionality in my OVN installation between v20.06.2 and latest branch-20.06 (78174ea) on cluster with enabled rbac. On v20.06.2 ovn-controller successfully claims port: # ovn-controller --version ovn-controller 20.06.2 Open vSwitch Library

[ovs-dev] ovs-kmod spec files

2018-07-06 Thread Odintsov Vladislav
Hi all, I'm trying to understand the needs to have openvswitch-kmod-fedora.spec and openvswitch-kmod-rhel6.spec simultaneously. Is it still possible to build master branch for el6 dists (CentOS/RH/OEL 6)? I'm going to make this PR up to day: https://github.com/openvswitch/ovs/pull/169 and wa