The following commit is required in order to fix ovn-controller
crash on port binding removal by northd:
  88e08a66e139 ("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 d512fda2b..88e08a66e 160000
--- a/ovs
+++ b/ovs
@@ -1 +1 @@
-Subproject commit d512fda2bffad6df5a13c1b5f4809f6a9dd7b70d
+Subproject commit 88e08a66e139f1b7c61d8e3c7e05e617d3dd2401
-- 
2.51.1

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

Reply via email to