The following commit is required in order to fix ovn-controller
crash on port binding removal by northd:
  785a89b48db6 ("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 f1bc9af96..785a89b48 160000
--- a/ovs
+++ b/ovs
@@ -1 +1 @@
-Subproject commit f1bc9af9627e97b8361dd32834068a8b7f2e0d17
+Subproject commit 785a89b48db6803a2ce44e28e76d8a0025be1803
-- 
2.51.1

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

Reply via email to