Re: [ovs-dev] [PATCH ovn v3] northd: Fix defrag flows for duplicate vips

2021-07-16 Thread Numan Siddique
On Fri, Jul 16, 2021 at 8:07 AM Mark Gray wrote: > > When adding two SB flows with the same vip but different protocols, only > the most recent flow will be added due to the `if` statement: > > if (!sset_contains(_ips, lb_vip->vip_str)) { > sset_add(_ips,

[ovs-dev] [PATCH ovn v3] northd: Fix defrag flows for duplicate vips

2021-07-16 Thread Mark Gray
When adding two SB flows with the same vip but different protocols, only the most recent flow will be added due to the `if` statement: if (!sset_contains(_ips, lb_vip->vip_str)) { sset_add(_ips, lb_vip->vip_str); This can cause unexpected behaviour when two load