On 10/6/25 12:18 PM, Ales Musil via dev wrote:
> This reverts commit 482af6847f9edbcc8834ab06a7d2d14c1dbe252c.
> 

I'd add a short description here with the reason for revert, e.g.:

"The change introduced a performance issue on northd restarts or
when standby northd process becomes active.  The cause is that
new northd instance would re-create every logical flow in Sb DB,
as it doesn't perform lookups for exisitng ones in the database."

> Reported-at: 
> ihttps://mail.openvswitch.org/pipermail/ovs-dev/2025-October/426727.html

nit: there is a stray 'i' in the link.

With these addressed, I think it's fine to revert for now and get
the proper solution when it is available:

Acked-by: Ilya Maximets <[email protected]>


> Signed-off-by: Ales Musil <[email protected]>
> ---
>  northd/en-lflow.c   |   2 +-
>  northd/lflow-mgr.c  | 108 ++++++++++++++++++++------------------------
>  northd/lflow-mgr.h  |  10 +---
>  northd/northd.c     |   8 ++--
>  tests/ovn-northd.at |  32 -------------
>  5 files changed, 54 insertions(+), 106 deletions(-)
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to