On Mon, Oct 6, 2025 at 6:20 PM Ilya Maximets <[email protected]> wrote:
> 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(-) > > Thank you Mark and Ilya, I have addressed the commit message, nit, added the missing Reported-by tag and merged this into main. Regards, Ales _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
