On 12/4/25 11:58 AM, Ales Musil wrote: > When multiple OVSDB updates have been received since the last northd run > it's possible that the IDL tracks changes for database entities that > were added _and also_ removed from the last time the northd processing > engine run. In some cases, those may appear as being simultaneously > "new" and "deleted". > > This might cause an issue when we try to update row, but end up > asserting on the row being invalid: > assertion row->new_datum != NULL failed in ovsdb_idl_txn_write__() > > To prevent that skip those entries that appear as both "new" and > "deleted" at the same time. > > Reported-at: https://issues.redhat.com/browse/FDP-2784 > Reported-by: Dumitru Ceara <[email protected]> > Signed-off-by: Ales Musil <[email protected]> > ---
Hi Ales, Thanks for the fix, it looks good to me! Maybe we should add a "Fixes" tag? I am running this through OVN-kubernetes CI (along with the other fixes from Xavier, Ilya and Mark) and we see no ovn-northd crashes until now: https://github.com/ovn-kubernetes/ovn-kubernetes/actions/runs/19923831779/ Assuming that stays the same until the job above finishes: Acked-by: Dumitru Ceara <[email protected]> Regards, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
