neutron-lbaas performance hit due to unnecessary lazy sql calls

[Erik Olof Gunnar Andersson]

In neutron-lbaas many ORM relationships are configured to be lazy joined. This is causing slow api calls.

The problem is that a call to get a load balancer or a list of load balancers, will recursively get all listeners, pools and members belonging to those load balancer(s). If lazy wasn't used, the initial sql query would instantly retrieve all objects belonging to that load balancer, but as lazy joined is used, it will initially only get the load balancer data, and later have to do additional sql calls to get the remaining objects.

It's worth mentioning that even if cli calls only show limited data on a load balancer, internally the same call would return data containing listeners, pools and members. Pretty much negating any improvements lazy joined queries brings to the table.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/477698
Committed: https://git.openstack.org/cgit/openstack/neutron-lbaas/commit/?id=7e35b183f925ad744cfcff5f405df131ad1d5f66
Submitter: Zuul
Branch: master

commit 7e35b183f925ad744cfcff5f405df131ad1d5f66
Author: Erik Olof Gunnar Andersson <email address hidden>
Date: Mon Jun 26 15:31:23 2017 -0700

    Remove unnecessary lazy-loaded queries

    This patch removes lazy-load queries to speed up database
    operations when using neutron-lbaas.

    Change-Id: I9d67f0966561baaefb50ae97b943ff6593e194eb
    Closes-Bug: #1706466

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/neutron-lbaas 12.0.0.0b3 development milestone.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/545796

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/545796
Committed: https://git.openstack.org/cgit/openstack/neutron-lbaas/commit/?id=606ec87a44162629ebfce4d279535124433b08fc
Submitter: Zuul
Branch: stable/pike

commit 606ec87a44162629ebfce4d279535124433b08fc
Author: Erik Olof Gunnar Andersson <email address hidden>
Date: Mon Jun 26 15:31:23 2017 -0700

    Remove unnecessary lazy-loaded queries

    This patch removes lazy-load queries to speed up database
    operations when using neutron-lbaas.

    Change-Id: I9d67f0966561baaefb50ae97b943ff6593e194eb
    Closes-Bug: #1706466
    (cherry picked from commit 7e35b183f925ad744cfcff5f405df131ad1d5f66)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/neutron-lbaas 11.0.3 release.

[Jan Zerebecki (jan-zerebecki)]

Follow up for this: https://review.openstack.org/#/c/568361/

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/ocata
Review: https://review.openstack.org/624158

[OpenStack Infra (hudson-openstack)]

Change abandoned by "Elod Illes <email address hidden>" on branch: stable/ocata
Review: https://review.opendev.org/c/openstack/neutron-lbaas/+/624158
Reason: neutron-lbaas's stable/ocata branch is tagged as End of Life (ocata-eol). To be able to delete the branch, every open patch need to be abandoned.