[ovs-dev] [PATCH ovn 1/2] Revert "northd: Don't poll ovsdb before the connection is fully established"

2021-09-16 Thread Zhen Wang via dev
From: zhen wang 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

Re: [ovs-dev] [PATCH ovn 1/2] Revert "northd: Don't poll ovsdb before the connection is fully established"

2021-09-17 Thread Numan Siddique
On Thu, Sep 16, 2021 at 11:05 PM Zhen Wang via dev wrote: > > From: zhen wang > > 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 po

Re: [ovs-dev] [PATCH ovn 1/2] Revert "northd: Don't poll ovsdb before the connection is fully established"

2021-09-17 Thread Han Zhou
On Thu, Sep 16, 2021 at 8:05 PM Zhen Wang wrote: > > From: zhen wang > > 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 > out

Re: [ovs-dev] [PATCH ovn 1/2] Revert "northd: Don't poll ovsdb before the connection is fully established"

2021-09-17 Thread Han Zhou
On Fri, Sep 17, 2021 at 1:31 PM Numan Siddique wrote: > > On Thu, Sep 16, 2021 at 11:05 PM Zhen Wang via dev > wrote: > > > > From: zhen wang > > > > This reverts commit 1e59feea933610b28fd4442243162ce35595cfee. > > Above commit introduced a bug when muptiple ovn-northd instances work in HA > >

Re: [ovs-dev] [PATCH ovn 1/2] Revert "northd: Don't poll ovsdb before the connection is fully established"

2021-09-17 Thread Renat Nurgaliyev
Hello Han, when I wrote this patch we had an issue with a very big SB database, around 1,5 gigabytes. There were no controllers or northds running, so the database server was without any load at all. Although OVSDB was idling, even a single northd process could not fully connect to the database du

Re: [ovs-dev] [PATCH ovn 1/2] Revert "northd: Don't poll ovsdb before the connection is fully established"

2021-09-17 Thread Han Zhou
On Fri, Sep 17, 2021 at 1:48 PM Renat Nurgaliyev wrote: > > Hello Han, > > when I wrote this patch we had an issue with a very big SB database, around 1,5 gigabytes. There were no controllers or northds running, so the database server was without any load at all. Although OVSDB was idling, even a

Re: [ovs-dev] [PATCH ovn 1/2] Revert "northd: Don't poll ovsdb before the connection is fully established"

2021-09-17 Thread Renat Nurgaliyev
Hi Han, yes, I believe you are totally right. But it still feels like a chicken and egg problem to me, storing the database timeout setting inside the database itself. If there would be at least some local command line argument to override timeout value, it would be already amazing, because curren

Re: [ovs-dev] [PATCH ovn 1/2] Revert "northd: Don't poll ovsdb before the connection is fully established"

2021-09-17 Thread Numan Siddique
On Fri, Sep 17, 2021 at 5:02 PM Renat Nurgaliyev wrote: > > Hi Han, > > yes, I believe you are totally right. But it still feels like a chicken and > egg problem to me, storing the database timeout setting inside the database > itself. If there would be at least some local command line argument to

Re: [ovs-dev] [PATCH ovn 1/2] Revert "northd: Don't poll ovsdb before the connection is fully established"

2021-09-17 Thread Renat Nurgaliyev
Hi Numan, according to our experience, in such case the connection can never be fully established, and northd ends up being in an endless loop with 100% CPU utilization. Numan Siddique schrieb am Sa., 18. Sept. 2021, 00:05: > On Fri, Sep 17, 2021 at 5:02 PM Renat Nurgaliyev > wrote: > > > > Hi

Re: [ovs-dev] [PATCH ovn 1/2] Revert "northd: Don't poll ovsdb before the connection is fully established"

2021-09-17 Thread Han Zhou
On Fri, Sep 17, 2021 at 2:04 PM Numan Siddique wrote: > > On Fri, Sep 17, 2021 at 5:02 PM Renat Nurgaliyev wrote: > > > > Hi Han, > > > > yes, I believe you are totally right. But it still feels like a chicken and > > egg problem to me, storing the database timeout setting inside the database > >

Re: [ovs-dev] [PATCH ovn 1/2] Revert "northd: Don't poll ovsdb before the connection is fully established"

2021-09-17 Thread Numan Siddique
On Fri, Sep 17, 2021 at 5:10 PM Han Zhou wrote: > > > > On Fri, Sep 17, 2021 at 2:04 PM Numan Siddique wrote: > > > > On Fri, Sep 17, 2021 at 5:02 PM Renat Nurgaliyev wrote: > > > > > > Hi Han, > > > > > > yes, I believe you are totally right. But it still feels like a chicken > > > and > > > e