Re: [ovs-dev] [PATCH ovn] ovn-controller: Don't enable conditional monitoring until connected.

On Tue, Aug 23, 2022 at 7:15 AM Han Zhou wrote: > > > > On Tue, Aug 23, 2022 at 6:45 AM Ilya Maximets wrote: > > > > On 8/22/22 19:39, Han Zhou wrote: > > > > > > > > > On Mon, Aug 22, 2022 at 3:33 AM Ilya Maximets mailto:i.maxim...@ovn.org>> wrote: > > >> > > >> On 8/22/22 12:21, Dumitru Ceara w

Re: [ovs-dev] [PATCH ovn] ovn-controller: Don't enable conditional monitoring until connected.

On Tue, Aug 23, 2022 at 6:45 AM Ilya Maximets wrote: > > On 8/22/22 19:39, Han Zhou wrote: > > > > > > On Mon, Aug 22, 2022 at 3:33 AM Ilya Maximets mailto:i.maxim...@ovn.org>> wrote: > >> > >> On 8/22/22 12:21, Dumitru Ceara wrote: > >> > Hi Ilya, > >> > > >> > On 8/22/22 11:19, Ilya Maximets wro

Re: [ovs-dev] [PATCH ovn] ovn-controller: Don't enable conditional monitoring until connected.

On 8/22/22 19:39, Han Zhou wrote: > > > On Mon, Aug 22, 2022 at 3:33 AM Ilya Maximets > wrote: >> >> On 8/22/22 12:21, Dumitru Ceara wrote: >> > Hi Ilya, >> > >> > On 8/22/22 11:19, Ilya Maximets wrote: >> >> daemon_started_recently() concept is flawed in terms that it

Re: [ovs-dev] [PATCH ovn] ovn-controller: Don't enable conditional monitoring until connected.

On Mon, Aug 22, 2022 at 3:33 AM Ilya Maximets wrote: > > On 8/22/22 12:21, Dumitru Ceara wrote: > > Hi Ilya, > > > > On 8/22/22 11:19, Ilya Maximets wrote: > >> daemon_started_recently() concept is flawed in terms that it uses > >> fixed number of iterations for a countdown and a fixed timeout, so

Re: [ovs-dev] [PATCH ovn] ovn-controller: Don't enable conditional monitoring until connected.

On 8/22/22 12:21, Dumitru Ceara wrote: > Hi Ilya, > > On 8/22/22 11:19, Ilya Maximets wrote: >> daemon_started_recently() concept is flawed in terms that it uses >> fixed number of iterations for a countdown and a fixed timeout, so >> the undesired removal of configured resources can still happen

Re: [ovs-dev] [PATCH ovn] ovn-controller: Don't enable conditional monitoring until connected.

Hi Ilya, On 8/22/22 11:19, Ilya Maximets wrote: > daemon_started_recently() concept is flawed in terms that it uses > fixed number of iterations for a countdown and a fixed timeout, so > the undesired removal of configured resources can still happen under > certain conditions even with this mechan

[ovs-dev] [PATCH ovn] ovn-controller: Don't enable conditional monitoring until connected.

daemon_started_recently() concept is flawed in terms that it uses fixed number of iterations for a countdown and a fixed timeout, so the undesired removal of configured resources can still happen under certain conditions even with this mechanism in place. The root cause of the original problem is