[ovs-dev] [PATCH ovn branch-21.09 0/3] Improve Load Balancer performance.

Numan Siddique numans at ovn.org
Tue Nov 9 16:29:18 UTC 2021


On Wed, Nov 3, 2021 at 4:11 PM Numan Siddique <numans at ovn.org> wrote:
>
> On Tue, Nov 2, 2021 at 3:35 PM Dumitru Ceara <dceara at redhat.com> wrote:
> >
> > On 11/2/21 8:30 PM, Dumitru Ceara wrote:
> > > This series backports the load balancer performance improvements to
> > > stable branch-21.09.
> > >
> > > - patch 1/3 changes the way ARP responder flows are generated for load
> > >   balancer VIPs, using an address set, making SB updates incremental
> > >   and also reducing memory usage on the SB server side.
> > > - patch 2/3 introduces a new feature, Load_Balancer_Group, which
> > >   simplifies load balancer configuration in large scale scenarios and
> > >   at the same time improves performance due to the large reduction in
> > >   database refereces between rows.
> > > - patch 3/3 is a follow up fix of a bug originally introduced by
> > >   patch 1/3.
> > >
> > > Usually new features and performance fixes are not to be backported to
> > > stable branches, however due to the (mis)alignment of upstream and
> > > downstream release schedules, and due to the fact that ovn-kubernetes
> > > requires a database table to be part of a schema that's tagged in a
> > > release in ovn-org/ovn repo, if load balancer groups would be available
> > > only starting with v21.12.0 then downstream ovn-kubernetes (and OpenShift)
> > > will not be able to consume the feature for two release cycles (~6 months).
> > >
> > > On the other hand the feature itself is quite contained, doesn't break
> > > backwards compatibility, and, when used, hugely improves some CMSs
> > > (OpenShift) load balancer use cases.
> > >
> > > Dumitru Ceara (3):
> > >       northd: Use address sets for ARP responder flows for VIPs.
> > >       nb: Add support for Load_Balancer_Groups.
> > >       northd: Always generate valid load balancer address set names.
> > >
> > >  NEWS                  |    2
> > >  northd/northd.c       |  334 +++++++++++++++++++++++++++++++++----------------
> > >  ovn-nb.ovsschema      |   24 +++-
> > >  ovn-nb.xml            |   37 +++++
> > >  tests/ovn-northd.at   |  293 ++++++++++++++++++++++++++++++++++---------
> > >  utilities/ovn-nbctl.c |    3
> > >  6 files changed, 517 insertions(+), 176 deletions(-)
> > >
> >
> > Adding OVN maintainers explicitly in order to discuss the possibility of
> > including this in a v21.09.1.
>
> +1 from me for the backports.

Thanks for the backport patches.  I went ahead and applied these
patches to branch-21.09.

I suppose the next step is to release 21.09.1

Numan

>
> Numan
>
> >
> > Regards,
> > Dumitru
> >
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >


More information about the dev mailing list