On Thu, Dec 4, 2025 at 5:22 PM Ilya Maximets <[email protected]> wrote:
> The following commit is required in order to fix ovn-controller
> crash on port binding removal by northd:
> 296465ccd16e ("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 2e7d3ea0d..296465ccd 160000
> --- a/ovs
> +++ b/ovs
> @@ -1 +1 @@
> -Subproject commit 2e7d3ea0d0cade43da9156646cf8260f454431e8
> +Subproject commit 296465ccd16e431dce703c9e7537d04ff0992b08
> --
> 2.51.1
>
>
Thank you Ilya,
I went ahead and merged this into 25.09.
Regards,
Ales
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
