Pagination limits code needs to be shared

[Vipul Sabhaya]

Currently, there is code in extensions/mysql/models.py that determines the limit to pass down to the guest. Similar code exists in instances/models.py.

    limit = cls.DEFAULT_LIMIT if limit > cls.DEFAULT_LIMIT else limit

We should have a common place where this logic can exist to ensure that behavior remains consistent.

[Amrith Kumar (amrith)]

Updating importance to "Wishlist" (bugscrub, 2014-10-23)

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/235017

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/235017
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=0746e70cb5e367d89980f4e1166458afed194fe1
Submitter: Jenkins
Branch: master

commit 0746e70cb5e367d89980f4e1166458afed194fe1
Author: Ishita Mandhan <email address hidden>
Date: Wed Oct 14 16:22:54 2015 -0700

    Pagination limit code refactor

    The logic to find the pagination limit is
    repeated several times and is not in one
    function. This patch creates a function that
    is called in various locations replacing
    repetitive code.

    Change-Id: I3eced6a4236e4189e57ea5a1589e8f0d411cc4e3
    Closes-Bug: #1308285

[Thierry Carrez (ttx)]

This issue was fixed in the openstack/trove 5.0.0.0b1 development milestone.