On Sat, Dec 5, 2020 at 1:10 AM Dumitru Ceara <dce...@redhat.com> wrote: > > On 12/4/20 7:50 PM, Ilya Maximets wrote: > > During refactoring, direct access such as 'lb->vips[i].backends[j].op' > > was simplified by the presence of the 'backend_nb' pointer. But instead > > of the inner one, the index of the outer loop was used. This forces > > northd to only use one backend per vip, and can also lead to invalid > > memory accesses. > > > > This change fixes system test: > > 24. ovn -- Load balancer health checks > > > > Fixes: f1119c125765 ("northd: Refactor load balancer vip parsing.") > > Tested-by: Lorenzo Bianconi <lorenzo.bianc...@redhat.com> > > Signed-off-by: Ilya Maximets <i.maxim...@ovn.org> > > --- > > Thanks for the fix! > > Acked-by: Dumitru Ceara <dce...@redhat.com>
Thanks Ilya for the fix and Dumitru for the review. I applied this patch to master and branch-20.12. Numan > > _______________________________________________ > dev mailing list > d...@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev