Connections and SSL columns are not used by ovn-northd. Omit them in IDL. This is not a functional problem, but it may hide incremental processing problems, because when status changes in Connection table, which is referenced by SB_Global, ovn-northd will receive notifications for SB_Global change, which triggers recompute. Some tests may pass when this happens, while it should have failed if the recompute were not triggered. An example is the test case "testing propagate Port_Binding.up to NB and OVS", which has passed in most cases but fails permanently afte this change (and will be fixed in a separate patch).
Signed-off-by: Han Zhou <[email protected]> --- northd/ovn-northd.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c index 0b8bbfb95cf7..647f60c8583f 100644 --- a/northd/ovn-northd.c +++ b/northd/ovn-northd.c @@ -818,6 +818,10 @@ main(int argc, char *argv[]) ovsdb_idl_track_add_all(ovnsb_idl_loop.idl); ovsdb_idl_set_write_changed_only_all(ovnsb_idl_loop.idl, true); + /* Omit unused columns. */ + ovsdb_idl_omit(ovnsb_idl_loop.idl, &sbrec_sb_global_col_connections); + ovsdb_idl_omit(ovnsb_idl_loop.idl, &sbrec_sb_global_col_ssl); + /* Disable alerting for pure write-only columns. */ ovsdb_idl_omit_alert(ovnsb_idl_loop.idl, &sbrec_sb_global_col_nb_cfg); ovsdb_idl_omit_alert(ovnsb_idl_loop.idl, &sbrec_address_set_col_name); -- 2.30.2 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
