On 12/8/25 9:55 AM, Xavier Simonart wrote:
> If a Logical Switch Port was added/updated and its Logical Switch deleted
> within the same northd iteration, ovn-northd was asserting [0].
>
> [0] lib/ovsdb-idl.c:3668: assertion row->new_datum != NULL failed in
> ovsdb_idl_txn_write__()
>
> Stack trace:
> 6 ovs_assert_failure lib/util.c:90
> 7 ovsdb_idl_txn_write__ lib/ovsdb-idl.c:3668
> 8 ovsdb_idl_txn_write__ lib/ovsdb-idl.c:3668
> 9 sbrec_port_binding_set_nat_addresses lib/ovn-sb-idl.c:40588
> 10 sync_pbs_for_northd_changed_ovn_ports northd/northd.c:3959
> 11 sync_to_sb_pb_northd_handler northd/en-sync-sb.c:420
> 12 engine_compute lib/inc-proc-eng.c:478
> 13 engine_run_node lib/inc-proc-eng.c:550
> 14 engine_run lib/inc-proc-eng.c:579
> 15 inc_proc_northd_run northd/inc-proc-northd.c:608
> 16 main northd/ovn-northd.c:1134
>
> Fixes: 06e2c1bf0ce7 ("northd: I-P for logical switch creation/deletion in
> en_northd.")
> Signed-off-by: Xavier Simonart <[email protected]>
>
> ---
> -v2: Updated based on Dumitru and Ilya's feedback i.e.
> -- Avoid change handlers changing input data.
> -- Do not use date +...%03N in tests as not portable and requiring GNU
> date.
> ---
> northd/northd.c | 127 +++++++++++++++++++++++---------------------
> tests/ovn-northd.at | 30 +++++++++++
> 2 files changed, 95 insertions(+), 62 deletions(-)
>
Hi Xavier,
Thanks for this new revision! Applied to main.
Regards,
Dumitru
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
