Bug 1388171

Summary:	[RFE] support new pagination mechanism for the nova simple-tenant-usage API
Product:	Red Hat OpenStack	Reporter:	Eoghan Glynn <eglynn>
Component:	python-django-horizon	Assignee:	Radomir Dopieralski <rdopiera>
Status:	CLOSED ERRATA	QA Contact:	Ola Pavlenko <opavlenk>
Severity:	unspecified	Docs Contact:
Priority:	high
Version:	11.0 (Ocata)	CC:	achernet, aortega, apannu, athomas, beth.white, ccollett, jrist, mrunge, nlevinki, opavlenk, pmyers, rdopiera, sclewis, srevivo
Target Milestone:	Upstream M3	Keywords:	FutureFeature, OtherQA, Triaged
Target Release:	11.0 (Ocata)
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	python-django-horizon-11.0.0-0.20170131043553.aa444a6.el7ost	Doc Type:	Enhancement
Doc Text:	To avoid memory bloat issues in the nova-api workers, pagination logic has been added to the simple-tenant-usage API extension.	Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-05-17 19:36:24 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1318250, 1426490

Description Eoghan Glynn 2016-10-24 16:09:09 UTC

To avoid memory bloat issues in the nova-api workers, the nova project is planning to add pagination logic to the simple-tenant-usage API extension in Ocata:

https://blueprints.launchpad.net/nova/+spec/paginate-simple-tenant-usage

In order to avoid continuing to DOS the nova-api tier, Horizon will need to support this new pagination scheme by traversing between pages via the next link, then subsequently stitching back up the response such that the server_usages list for each tenant is made whole (such lists may cross page boundaries under the proposed solution).

Comment 1 Radomir Dopieralski 2016-11-03 16:18:34 UTC

Notes for using the API:

https://github.com/dianaclarke/openstack-notes/wiki/simple%20tenant%20usage

Places that will probably need changing:

https://github.com/openstack/horizon/blob/92972d4d1483dc3876416a9269ac529d43537980/openstack_dashboard/api/nova.py#L164
https://github.com/openstack/horizon/blob/92972d4d1483dc3876416a9269ac529d43537980/openstack_dashboard/usage/base.py#L252
https://github.com/openstack/horizon/blob/92972d4d1483dc3876416a9269ac529d43537980/openstack_dashboard/usage/base.py#L259

Upstream blueprint:

https://blueprints.launchpad.net/horizon/+spec/simple-tenant-usage-pagination

Comment 2 Radomir Dopieralski 2016-12-14 11:04:46 UTC

A draft patch https://review.openstack.org/#/c/410337/

We still need to resolve a problem with specifying the API version https://review.openstack.org/#/c/410688/ and make Horizon load the correct microversion of the Nova API.

We also need to write tests for all this.

Comment 14 Radomir Dopieralski 2017-04-28 12:19:34 UTC

This feature is not visible to the users and as such can't be easily tested. The improvements in performance are not visible in small scale deployments.

Comment 16 errata-xmlrpc 2017-05-17 19:36:24 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2017:1245