[ovs-dev] [PATCH ovn v2] ovn-controller: Reduce size of the SB monitor condition.

2023-06-21 Thread Dumitru Ceara
We don't need to explicitly add port bindings that were already bound locally. We implicitly get those because we monitor the datapaths they're attached to. When performing an ovn-heater 500-node density-heavy scale test [0], with conditional monitoring enabled, the unreasonably long poll interva

Re: [ovs-dev] [PATCH ovn v2] ovn-controller: Reduce size of the SB monitor condition.

2023-06-21 Thread Han Zhou
On Wed, Jun 21, 2023 at 2:48 AM Dumitru Ceara wrote: > > We don't need to explicitly add port bindings that were already bound > locally. We implicitly get those because we monitor the datapaths > they're attached to. > > When performing an ovn-heater 500-node density-heavy scale test [0], with >

Re: [ovs-dev] [PATCH ovn v2] ovn-controller: Reduce size of the SB monitor condition.

2023-06-22 Thread Dumitru Ceara
On 6/21/23 16:50, Han Zhou wrote: > On Wed, Jun 21, 2023 at 2:48 AM Dumitru Ceara wrote: >> >> We don't need to explicitly add port bindings that were already bound >> locally. We implicitly get those because we monitor the datapaths >> they're attached to. >> >> When performing an ovn-heater 500

Re: [ovs-dev] [PATCH ovn v2] ovn-controller: Reduce size of the SB monitor condition.

2023-06-27 Thread Han Zhou
On Thu, Jun 22, 2023 at 1:12 AM Dumitru Ceara wrote: > > On 6/21/23 16:50, Han Zhou wrote: > > On Wed, Jun 21, 2023 at 2:48 AM Dumitru Ceara wrote: > >> > >> We don't need to explicitly add port bindings that were already bound > >> locally. We implicitly get those because we monitor the datapat

Re: [ovs-dev] [PATCH ovn v2] ovn-controller: Reduce size of the SB monitor condition.

2023-06-28 Thread Dumitru Ceara
On 6/28/23 04:54, Han Zhou wrote: > On Thu, Jun 22, 2023 at 1:12 AM Dumitru Ceara wrote: >> >> On 6/21/23 16:50, Han Zhou wrote: >>> On Wed, Jun 21, 2023 at 2:48 AM Dumitru Ceara wrote: We don't need to explicitly add port bindings that were already bound locally. We implicitly ge

Re: [ovs-dev] [PATCH ovn v2] ovn-controller: Reduce size of the SB monitor condition.

2023-06-28 Thread Ilya Maximets
On 6/28/23 10:30, Dumitru Ceara wrote: > On 6/28/23 04:54, Han Zhou wrote: >> On Thu, Jun 22, 2023 at 1:12 AM Dumitru Ceara wrote: >>> >>> On 6/21/23 16:50, Han Zhou wrote: On Wed, Jun 21, 2023 at 2:48 AM Dumitru Ceara wrote: > > We don't need to explicitly add port bindings that wer

Re: [ovs-dev] [PATCH ovn v2] ovn-controller: Reduce size of the SB monitor condition.

2023-06-28 Thread Dumitru Ceara
On 6/28/23 12:20, Ilya Maximets wrote: > On 6/28/23 10:30, Dumitru Ceara wrote: >> On 6/28/23 04:54, Han Zhou wrote: >>> On Thu, Jun 22, 2023 at 1:12 AM Dumitru Ceara wrote: On 6/21/23 16:50, Han Zhou wrote: > On Wed, Jun 21, 2023 at 2:48 AM Dumitru Ceara wrote: >> >> We don