On Thu, Sep 16, 2021 at 11:05 PM Zhen Wang via dev <ovs-dev@openvswitch.org> wrote: > > From: zhen wang <zhew...@nvidia.com> > > This reverts commit 1e59feea933610b28fd4442243162ce35595cfee. > Above commit introduced a bug when muptiple ovn-northd instances work in HA > mode. If SB leader and active ovn-northd instance got killed by system power > outage, standby ovn-northd instance would never detect the failure. > > Signed-off-by: zhen wang <zhew...@nvidia.com>
I'm not sure if we can revert this commit. How would you propose to fix the issue mentioned in the commit message of the commit 1e59feea933610 ? I think you should configure a desired probe interval value in NB_Global.options so that the stand-by northd instance can detect the failure. With this configuration and with your patch 2, the issue reported by you should be addressed. Another option is to reset the probe interval from 0 to the default value - DEFAULT_PROBE_INTERVAL_MSEC once northd has successfully connected to the Northbound db server if CMS has not configured the probe interval in NB_Global.options. Thanks Numan > --- > northd/northd.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/northd/northd.c b/northd/northd.c > index 688a6e4ef..b7e64470f 100644 > --- a/northd/northd.c > +++ b/northd/northd.c > @@ -74,8 +74,8 @@ static bool use_ct_inv_match = true; > > /* Default probe interval for NB and SB DB connections. */ > #define DEFAULT_PROBE_INTERVAL_MSEC 5000 > -static int northd_probe_interval_nb = 0; > -static int northd_probe_interval_sb = 0; > +static int northd_probe_interval_nb = DEFAULT_PROBE_INTERVAL_MSEC; > +static int northd_probe_interval_sb = DEFAULT_PROBE_INTERVAL_MSEC; > #define MAX_OVN_TAGS 4096 > > /* Pipeline stages. */ > -- > 2.20.1 > > _______________________________________________ > dev mailing list > d...@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev