Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-05-02 Thread Vladislav Odintsov
Hi Dumitru, I’m a bit concerned about our last conversation’s idea how to set inactivity probe. The point is that in general case dbctl module can be used by multiple existing and possible future ovn-*ctl utilities. So, to configure a basic (we may call it a "startup" idl inactivity probe), whi

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-04-18 Thread Vladislav Odintsov
> On 18 Apr 2023, at 16:59, Dumitru Ceara wrote: > > On 4/14/23 15:37, Vladislav Odintsov wrote: >> Hi Dumitru, >> >>> On 13 Apr 2023, at 11:55, Dumitru Ceara wrote: >>> >>> Hi Vladislav, >>> >>> On 4/3/23 12:08, Ilya Maximets wrote: On 3/31/23 17:17, Dumitru Ceara wrote: > On 3/31

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-04-18 Thread Dumitru Ceara
On 4/14/23 15:37, Vladislav Odintsov wrote: > Hi Dumitru, > >> On 13 Apr 2023, at 11:55, Dumitru Ceara wrote: >> >> Hi Vladislav, >> >> On 4/3/23 12:08, Ilya Maximets wrote: >>> On 3/31/23 17:17, Dumitru Ceara wrote: On 3/31/23 16:51, Vladislav Odintsov wrote: > As I understood from Ilya

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-04-14 Thread Vladislav Odintsov
Hi Dumitru, > On 13 Apr 2023, at 11:55, Dumitru Ceara wrote: > > Hi Vladislav, > > On 4/3/23 12:08, Ilya Maximets wrote: >> On 3/31/23 17:17, Dumitru Ceara wrote: >>> On 3/31/23 16:51, Vladislav Odintsov wrote: As I understood from Ilya, in case of one-command run of ovn-sbctl (non-d

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-04-13 Thread Dumitru Ceara
Hi Vladislav, On 4/3/23 12:08, Ilya Maximets wrote: > On 3/31/23 17:17, Dumitru Ceara wrote: >> On 3/31/23 16:51, Vladislav Odintsov wrote: >>> As I understood from Ilya, in case of one-command run of ovn-sbctl >>> (non-daemon mode), it doesn’t make sense to have client -> server >>> inactivity

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-04-03 Thread Ilya Maximets
On 3/31/23 17:17, Dumitru Ceara wrote: > On 3/31/23 16:51, Vladislav Odintsov wrote: >> As I understood from Ilya, in case of one-command run of ovn-sbctl >> (non-daemon mode), it doesn’t make sense to have client -> server inactivity >> probes. If cable is unplugged, will just hit tcp session ti

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-03-31 Thread Dumitru Ceara
On 3/31/23 16:51, Vladislav Odintsov wrote: > As I understood from Ilya, in case of one-command run of ovn-sbctl > (non-daemon mode), it doesn’t make sense to have client -> server inactivity > probes. If cable is unplugged, will just hit tcp session timeout, IIUC. > Please correct me if I’m wron

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-03-31 Thread Vladislav Odintsov
As I understood from Ilya, in case of one-command run of ovn-sbctl (non-daemon mode), it doesn’t make sense to have client -> server inactivity probes. If cable is unplugged, will just hit tcp session timeout, IIUC. Please correct me if I’m wrong. > On 31 Mar 2023, at 15:55, Dumitru Ceara wrote

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-03-31 Thread Dumitru Ceara
On 3/31/23 14:46, Vladislav Odintsov wrote: > Hi Dumitru, > >> On 31 Mar 2023, at 15:01, Dumitru Ceara wrote: >> >> On 3/31/23 11:43, Ales Musil wrote: >>> On Thu, Mar 23, 2023 at 8:25 PM Vladislav Odintsov >>> wrote: >>> For large OVN_Southbound DBs defatult interval of 5000 ms could be no

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-03-31 Thread Vladislav Odintsov
Hi Dumitru, > On 31 Mar 2023, at 15:01, Dumitru Ceara wrote: > > On 3/31/23 11:43, Ales Musil wrote: >> On Thu, Mar 23, 2023 at 8:25 PM Vladislav Odintsov >> wrote: >> >>> For large OVN_Southbound DBs defatult interval of 5000 ms could be not >>> sufficient. This patch disables OVSDB inactivi

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-03-31 Thread Dumitru Ceara
On 3/31/23 11:43, Ales Musil wrote: > On Thu, Mar 23, 2023 at 8:25 PM Vladislav Odintsov > wrote: > >> For large OVN_Southbound DBs defatult interval of 5000 ms could be not >> sufficient. This patch disables OVSDB inactivity probes for ovn-*ctl >> running >> in non-daemon mode. >> >> Signed-off

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-03-31 Thread Ales Musil
On Thu, Mar 23, 2023 at 8:25 PM Vladislav Odintsov wrote: > For large OVN_Southbound DBs defatult interval of 5000 ms could be not > sufficient. This patch disables OVSDB inactivity probes for ovn-*ctl > running > in non-daemon mode. > > Signed-off-by: Vladislav Odintsov > --- > utilities/ovn-

[ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-03-23 Thread Vladislav Odintsov
For large OVN_Southbound DBs defatult interval of 5000 ms could be not sufficient. This patch disables OVSDB inactivity probes for ovn-*ctl running in non-daemon mode. Signed-off-by: Vladislav Odintsov --- utilities/ovn-dbctl.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/utilities/ov