On Thu, Dec 4, 2025 at 12:15 PM Dumitru Ceara <[email protected]> wrote:
> 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 > > Thank you Dumitru, I have added the fixes tag, went ahead and merged this into main and backported all the way down to 24.03. Regards, Ales _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
