On Thu, Dec 4, 2025 at 5:24 PM Ilya Maximets <[email protected]> wrote:
> The following commit is required in order to fix ovn-controller
> crash on port binding removal by northd:
> ea3667c8931a ("ovsdb-idl: Fix returning non-existent rows from uuid
> lookup.")
>
> Without the change, ovn-controller may try to update an already deleted
> row causing assertion failure:
>
> 5 vlog_abort at lib/vlog.c:1325
> 6 ovs_assert_failure at lib/util.c:90
> 7 ovsdb_idl_txn_write__.constprop.0 at lib/ovsdb-idl.c:3650
> 8 ovsdb_idl_txn_write at lib/ovsdb-idl.c:3742
> 9 sbrec_port_binding_set_up at lib/ovn-sb-idl.c:39665
> 10 port_binding_set_down at controller/binding.c:3700
> 11 if_status_mgr_update at controller/if-status.c:645
> 12 main at controller/ovn-controller.c:7544
>
> 7 ovsdb_idl_txn_write__.constprop.0 at lib/ovsdb-idl.c:3650
> 3650 ovs_assert(row->new_datum != NULL);
>
> Reported-at: https://issues.redhat.com/browse/FDP-2807
> Reported-by: Dumitru Ceara <[email protected]>
> Signed-off-by: Ilya Maximets <[email protected]>
> ---
> ovs | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ovs b/ovs
> index 0c631d71e..ea3667c89 160000
> --- a/ovs
> +++ b/ovs
> @@ -1 +1 @@
> -Subproject commit 0c631d71e69d22fd27dc2e1db72b0c786c4de51c
> +Subproject commit ea3667c8931afbea902fed2aa7255d0f6ed890c0
> --
> 2.51.1
>
>
Thank you Ilya,
I went ahead and merged this into 24.09.
Regards,
Ales
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
