[ovs-dev] [PATCH ovn] ct-zones: reserve zones for upcoming ports

2024-07-22 Thread Mansi Sharma
This change will be useful for migration cases, where it can be used to sync ct-entries before port is up on new chassis, resulting in reduced network package drops. It also fulfills the need of any other service which might need advance ct-zone reservation in future. Signed-off-by: Mansi Sharma

Re: [ovs-dev] [PATCH ovn] controller: Add lsp option disable_garp_rarp.

2024-07-22 Thread Shibir Basak
Hi Numan, The solution you are suggesting looks fine for the ports to which IP management is not done by OVN. If OVN is acting as DHCP server then, I can think of following issues. 1. There will be a small window between the point when migration is complete and mac/ip is configured. During t

[ovs-dev] [PATCH] netdev-dpdk: Fix race condition in mempool information dump.

2024-07-22 Thread Mike Pattrick
Currently it is possible to call netdev-dpdk/get-mempool-info before a mempool as been created. This can happen because a device is added to the netdev_shash before a mempool is allocated for it, which results in a segmentation fault. Now we check for a NULL value before attempting to dereference

Re: [ovs-dev] [ovs-build] |fail| pw1960612 [ovs-dev, 2/2] Prepare for post-3.4.0 (3.4.90).

2024-07-22 Thread Ilya Maximets
On 7/19/24 14:35, Phelan, Michael wrote: > >> -Original Message- >> From: Ilya Maximets >> Sent: Thursday, July 18, 2024 9:09 PM >> To: Phelan, Michael >> Cc: i.maxim...@ovn.org; ovs-dev >> Subject: Re: [ovs-build] |fail| pw1960612 [ovs-dev,2/2] Prepare for post- >> 3.4.0 (3.4.90). >> >

Re: [ovs-dev] [PATCH ovn] northd: Fix issues for Forwarding_Group

2024-07-22 Thread Numan Siddique
On Fri, Jul 12, 2024 at 6:34 AM 1130911995--- via dev wrote: > > From 805422a85cf839bdff5330cd173f3ae55861efe9 Mon Sep 17 00:00:00 2001 > From: zhangqiang45 > Date: Mon, 8 Jul 2024 14:25:04 +0800 > Subject: [PATCH ovn] northd:Fix issues for Forwarding_Group The use of > variables from the outer

Re: [ovs-dev] [PATCH v7 ovn] northd: Add bfd, static_routes and route_policies nodes to I-P engine.

2024-07-22 Thread Numan Siddique
On Mon, Jul 22, 2024 at 2:04 AM Han Zhou wrote: > > On Sat, Jul 20, 2024 at 8:22 AM Lorenzo Bianconi > wrote: > > > > > On Fri, Jul 19, 2024 at 9:20 AM Lorenzo Bianconi > > > wrote: > > > > > > > > [...] > > > > > > > > > > Hi Han, > > > > > > > > > > I had suggested Lorenzo to add a new node to

[ovs-dev] [PATCH] ovsdb: transaction: Remove incorrect transaction abort in pre-commit.

2024-07-22 Thread Ilya Maximets
Pre-commit must not abort the transaction, otherwise the upper layers may crash accessing it. E.g. ovsdb_trigger_try() checks the state of the transaction after trying to commit it. This particular failure can't actually happen, because the function determine_changes() can't fail. However, the c

Re: [ovs-dev] [Patch ovn v2] actions: Explicitly finish CT actions.

2024-07-22 Thread martin . kalcok
Regarding the failed CI run, I suspect that there might be some test instability. All tests passed successfully in my branch [0] for this patch. Martin. [0] https://github.com/mkalcok/ovn/actions/runs/10009002777 On Fri, 2024-07-19 at 15:37 +0200, Martin Kalcok wrote: > As discussed here [0], a