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

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to