From 5efdf010b58134e2541b5d02bf8f23b47147d17c Mon Sep 17 00:00:00 2001 From: Dumitru Ceara Date: Tue, 10 Oct 2023 17:20:20 +0200 Subject: [PATCH] northd: Reset ls_datapath_group if not all chassis support it. The correct upgrade procedure mentions that databases are upgraded at the same time as ovn-northd, so there's nothing that doesn't allow northd to write to new columns. The feature flag reports instead what features chassis are aware of. In this specific case, if a chassis doesn't know how to interpret the 'ls_datapaths' column, ovn-northd should set it to NULL and use the deprecated 'datapath' one. Fixes: ab7a8fe4c018 ("northd: introduce ls_datapath_group column in lb sb db table") Signed-off-by: Dumitru Ceara Acked-by: Ales Musil --- northd/northd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/northd/northd.c b/northd/northd.c index 92052a9669..916068d44a 100644 --- a/northd/northd.c +++ b/northd/northd.c @@ -4678,6 +4678,7 @@ sync_lbs(struct ovsdb_idl_txn *ovnsb_txn, sbrec_load_balancer_set_datapath_group(sbrec_lb, NULL); } else { /* datapath_group column is deprecated. */ + sbrec_load_balancer_set_ls_datapath_group(sbrec_lb, NULL); sbrec_load_balancer_set_datapath_group( sbrec_lb, lb_dpg ? lb_dpg->dp_group : NULL );