Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-06 Thread Numan Siddique
On Tue, Feb 6, 2024 at 3:52 PM Han Zhou wrote: > > On Mon, Feb 5, 2024 at 7:47 PM Numan Siddique wrote: > > > > On Mon, Feb 5, 2024 at 9:41 PM Han Zhou wrote: > > > > > > On Mon, Feb 5, 2024 at 4:12 PM Numan Siddique wrote: > > > > > > > > On Mon, Feb 5, 2024 at 5:54 PM Han Zhou wrote: > > >

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-06 Thread Ilya Maximets
On 2/6/24 22:22, Ilya Maximets wrote: >>> >>> I did some testing with my patch and these are the findings >>> >>>                                      | Avg. Poll Intervals | Total >>> test time |  northd RSS >>> -- ++--- >>> Last

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-06 Thread Ilya Maximets
>> >> I did some testing with my patch and these are the findings >> >>                                      | Avg. Poll Intervals | Total >> test time |  northd RSS >> -- ++--- >> Last week                     |      1.5 seconds    

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-06 Thread Han Zhou
On Mon, Feb 5, 2024 at 7:47 PM Numan Siddique wrote: > > On Mon, Feb 5, 2024 at 9:41 PM Han Zhou wrote: > > > > On Mon, Feb 5, 2024 at 4:12 PM Numan Siddique wrote: > > > > > > On Mon, Feb 5, 2024 at 5:54 PM Han Zhou wrote: > > > > > > > > On Mon, Feb 5, 2024 at 10:15 AM Ilya Maximets > >

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-06 Thread Dumitru Ceara
On 2/5/24 23:53, Han Zhou wrote: > For Dumitru's proposal: >>> Maybe it would be an idea to integrate some of Han's performance testing >>> scripts into the set of tests we already have in the upstream repo, >>> ovn-performance.at [0], and run those in GitHub actions too. >>> [0]

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-05 Thread Numan Siddique
On Mon, Feb 5, 2024 at 9:41 PM Han Zhou wrote: > > On Mon, Feb 5, 2024 at 4:12 PM Numan Siddique wrote: > > > > On Mon, Feb 5, 2024 at 5:54 PM Han Zhou wrote: > > > > > > On Mon, Feb 5, 2024 at 10:15 AM Ilya Maximets > wrote: > > > > > > > > On 2/5/24 15:45, Ilya Maximets wrote: > > > > > On

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-05 Thread Han Zhou
On Mon, Feb 5, 2024 at 4:12 PM Numan Siddique wrote: > > On Mon, Feb 5, 2024 at 5:54 PM Han Zhou wrote: > > > > On Mon, Feb 5, 2024 at 10:15 AM Ilya Maximets wrote: > > > > > > On 2/5/24 15:45, Ilya Maximets wrote: > > > > On 2/5/24 11:34, Ilya Maximets wrote: > > > >> On 2/5/24 09:23, Dumitru

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-05 Thread Numan Siddique
On Mon, Feb 5, 2024 at 5:54 PM Han Zhou wrote: > > On Mon, Feb 5, 2024 at 10:15 AM Ilya Maximets wrote: > > > > On 2/5/24 15:45, Ilya Maximets wrote: > > > On 2/5/24 11:34, Ilya Maximets wrote: > > >> On 2/5/24 09:23, Dumitru Ceara wrote: > > >>> On 2/5/24 08:13, Han Zhou wrote: > > On Sun,

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-05 Thread Han Zhou
On Mon, Feb 5, 2024 at 10:15 AM Ilya Maximets wrote: > > On 2/5/24 15:45, Ilya Maximets wrote: > > On 2/5/24 11:34, Ilya Maximets wrote: > >> On 2/5/24 09:23, Dumitru Ceara wrote: > >>> On 2/5/24 08:13, Han Zhou wrote: > On Sun, Feb 4, 2024 at 9:26 PM Numan Siddique wrote: > > > >

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-05 Thread Ilya Maximets
On 2/5/24 15:45, Ilya Maximets wrote: > On 2/5/24 11:34, Ilya Maximets wrote: >> On 2/5/24 09:23, Dumitru Ceara wrote: >>> On 2/5/24 08:13, Han Zhou wrote: On Sun, Feb 4, 2024 at 9:26 PM Numan Siddique wrote: > > On Sun, Feb 4, 2024 at 9:53 PM Han Zhou wrote: >> >> On Sun,

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-05 Thread Dumitru Ceara
On 2/5/24 12:10, Ilya Maximets wrote: > On 2/5/24 11:58, Dumitru Ceara wrote: >> On 2/5/24 11:34, Ilya Maximets wrote: >>> On 2/5/24 09:23, Dumitru Ceara wrote: On 2/5/24 08:13, Han Zhou wrote: > On Sun, Feb 4, 2024 at 9:26 PM Numan Siddique wrote: >> >> On Sun, Feb 4, 2024 at

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-05 Thread Ilya Maximets
On 2/5/24 11:34, Ilya Maximets wrote: > On 2/5/24 09:23, Dumitru Ceara wrote: >> On 2/5/24 08:13, Han Zhou wrote: >>> On Sun, Feb 4, 2024 at 9:26 PM Numan Siddique wrote: On Sun, Feb 4, 2024 at 9:53 PM Han Zhou wrote: > > On Sun, Feb 4, 2024 at 5:46 AM Ilya Maximets wrote:

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-05 Thread Ilya Maximets
On 2/5/24 11:58, Dumitru Ceara wrote: > On 2/5/24 11:34, Ilya Maximets wrote: >> On 2/5/24 09:23, Dumitru Ceara wrote: >>> On 2/5/24 08:13, Han Zhou wrote: On Sun, Feb 4, 2024 at 9:26 PM Numan Siddique wrote: > > On Sun, Feb 4, 2024 at 9:53 PM Han Zhou wrote: >> >> On Sun,

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-05 Thread Dumitru Ceara
On 2/5/24 11:34, Ilya Maximets wrote: > On 2/5/24 09:23, Dumitru Ceara wrote: >> On 2/5/24 08:13, Han Zhou wrote: >>> On Sun, Feb 4, 2024 at 9:26 PM Numan Siddique wrote: On Sun, Feb 4, 2024 at 9:53 PM Han Zhou wrote: > > On Sun, Feb 4, 2024 at 5:46 AM Ilya Maximets wrote:

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-05 Thread Ilya Maximets
On 2/5/24 09:23, Dumitru Ceara wrote: > On 2/5/24 08:13, Han Zhou wrote: >> On Sun, Feb 4, 2024 at 9:26 PM Numan Siddique wrote: >>> >>> On Sun, Feb 4, 2024 at 9:53 PM Han Zhou wrote: On Sun, Feb 4, 2024 at 5:46 AM Ilya Maximets wrote: > > 35 files changed, 9681

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-05 Thread Dumitru Ceara
On 2/5/24 08:13, Han Zhou wrote: > On Sun, Feb 4, 2024 at 9:26 PM Numan Siddique wrote: >> >> On Sun, Feb 4, 2024 at 9:53 PM Han Zhou wrote: >>> >>> On Sun, Feb 4, 2024 at 5:46 AM Ilya Maximets wrote: >>> 35 files changed, 9681 insertions(+), 4645 deletions(-) >>> >>>

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-04 Thread Han Zhou
On Sun, Feb 4, 2024 at 9:26 PM Numan Siddique wrote: > > On Sun, Feb 4, 2024 at 9:53 PM Han Zhou wrote: > > > > On Sun, Feb 4, 2024 at 5:46 AM Ilya Maximets wrote: > > > > > > >>> > > > >>> > 35 files changed, 9681 insertions(+), 4645 deletions(-) > > > >>> > > > >>> I had another look at this

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-04 Thread Numan Siddique
On Sun, Feb 4, 2024 at 9:53 PM Han Zhou wrote: > > On Sun, Feb 4, 2024 at 5:46 AM Ilya Maximets wrote: > > > > >>> > > >>> > 35 files changed, 9681 insertions(+), 4645 deletions(-) > > >>> > > >>> I had another look at this series and acked the remaining patches. I > > >>> just had some minor

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-04 Thread Han Zhou
On Sun, Feb 4, 2024 at 5:46 AM Ilya Maximets wrote: > > >>> > >>> > 35 files changed, 9681 insertions(+), 4645 deletions(-) > >>> > >>> I had another look at this series and acked the remaining patches. I > >>> just had some minor comments that can be easily fixed when applying the > >>>

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-04 Thread Ilya Maximets
>>> >>> > 35 files changed, 9681 insertions(+), 4645 deletions(-) >>> >>> I had another look at this series and acked the remaining patches. I >>> just had some minor comments that can be easily fixed when applying the >>> patches to the main branch. >>> >>> Thanks for all the work on this! It

Re: [ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-02-02 Thread Dumitru Ceara
On 1/30/24 22:20, num...@ovn.org wrote: > From: Numan Siddique > Hi Numan, > This patch series adds incremental processing in the lflow engine > node to handle changes to northd and other engine nodes. > Changed related to load balancers and NAT are mainly handled in > this patch series. > >

[ovs-dev] [PATCH ovn v6 00/13] northd lflow incremental processing

2024-01-30 Thread numans
From: Numan Siddique This patch series adds incremental processing in the lflow engine node to handle changes to northd and other engine nodes. Changed related to load balancers and NAT are mainly handled in this patch series. This patch series can also be found here -