Re: [ovs-dev] [PATCH ovn v2 2/2] northd: Add I-P for syncing SB address sets.

2022-11-15 Thread Numan Siddique
On Tue, Nov 15, 2022 at 11:10 AM Mark Michelson wrote: > > On 11/14/22 16:34, Numan Siddique wrote: > > On Mon, Nov 14, 2022 at 3:23 PM Mark Michelson wrote: > >> > >> Hi Numan, I have just one minor suggestion below. > >> > >> On 11/14/22 11:48, num...@ovn.org wrote: > >>> From: Numan Siddique

Re: [ovs-dev] [PATCH ovn v2 2/2] northd: Add I-P for syncing SB address sets.

2022-11-15 Thread Mark Michelson
On 11/14/22 16:34, Numan Siddique wrote: On Mon, Nov 14, 2022 at 3:23 PM Mark Michelson wrote: Hi Numan, I have just one minor suggestion below. On 11/14/22 11:48, num...@ovn.org wrote: From: Numan Siddique Updates to NB address sets and NB port groups are handled incrementally for syncing

Re: [ovs-dev] [PATCH ovn v2 2/2] northd: Add I-P for syncing SB address sets.

2022-11-14 Thread Numan Siddique
On Mon, Nov 14, 2022 at 3:23 PM Mark Michelson wrote: > > Hi Numan, I have just one minor suggestion below. > > On 11/14/22 11:48, num...@ovn.org wrote: > > From: Numan Siddique > > > > Updates to NB address sets and NB port groups are handled > > incrementally for syncing the SB address sets. T

Re: [ovs-dev] [PATCH ovn v2 2/2] northd: Add I-P for syncing SB address sets.

2022-11-14 Thread Mark Michelson
Hi Numan, I have just one minor suggestion below. On 11/14/22 11:48, num...@ovn.org wrote: From: Numan Siddique Updates to NB address sets and NB port groups are handled incrementally for syncing the SB address sets. This patch doesn't support syncing the SB Address sets for the router load b

[ovs-dev] [PATCH ovn v2 2/2] northd: Add I-P for syncing SB address sets.

2022-11-14 Thread numans
From: Numan Siddique Updates to NB address sets and NB port groups are handled incrementally for syncing the SB address sets. This patch doesn't support syncing the SB Address sets for the router load balancer vips incrementally, instead a full recompute is triggered for any changes to NB load b