Bug 1192162 - Content View Versions api should be paginated but isn't (performance issue)
Summary: Content View Versions api should be paginated but isn't (performance issue)
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: API
Version: 6.1.0
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: Unspecified
Assignee: David Davis
QA Contact: Tazim Kolhar
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-02-12 17:57 UTC by Justin Sherrill
Modified: 2017-02-23 20:31 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-08-12 13:59:06 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 10014 0 None None None 2016-04-22 16:40:49 UTC

Description Justin Sherrill 2015-02-12 17:57:42 UTC
Description of problem:

Currently the content view versions api (which is used by the content view versions list) is not paged.  This leads to performance issues with lots of versions.

Version-Release number of selected component (if applicable):


How reproducible:
Always


Steps to Reproduce:
1. Try to fetch a list of content view versions for a content view with over 50 content views

Actual results:
Get all versions and may take a while


Expected results:
Should be 'fast' and only return one pages worth of data (like all other apis)


Additional info:

related to https://bugzilla.redhat.com/show_bug.cgi?id=1161636

Comment 2 Partha Aji 2015-04-02 21:20:03 UTC
Created redmine issue http://projects.theforeman.org/issues/10014 from this bug

Comment 4 Bryan Kearney 2015-05-20 10:07:18 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/10014 has been closed
-------------
David Davis
Applied in changeset commit:katello|15b8f9f131fe023f47fc19d584f83a202006e18f.

Comment 7 Tazim Kolhar 2015-06-09 08:52:41 UTC
VERIFIED:

# rpm -qa | grep foreman
ruby193-rubygem-foreman-tasks-0.6.12.7-1.el7sat.noarch
rubygem-hammer_cli_foreman_docker-0.0.3.6-1.el7sat.noarch
foreman-libvirt-1.7.2.26-1.el7sat.noarch
ruby193-rubygem-foreman_gutterball-0.0.1.9-1.el7sat.noarch
ruby193-rubygem-foreman_bootdisk-4.0.2.13-1.el7sat.noarch
ibm-x3755-02.ovirt.rhts.eng.bos.redhat.com-foreman-proxy-1.0-1.noarch
foreman-gce-1.7.2.26-1.el7sat.noarch
rubygem-hammer_cli_foreman_discovery-0.0.1.10-1.el7sat.noarch
foreman-selinux-1.7.2.13-1.el7sat.noarch
foreman-compute-1.7.2.26-1.el7sat.noarch
foreman-ovirt-1.7.2.26-1.el7sat.noarch
rubygem-hammer_cli_foreman-0.1.4.14-1.el7sat.noarch
foreman-postgresql-1.7.2.26-1.el7sat.noarch
ruby193-rubygem-foreman_docker-1.2.0.14-1.el7sat.noarch
ruby193-rubygem-foreman_discovery-2.0.0.15-1.el7sat.noarch
ruby193-rubygem-foreman-redhat_access-0.2.0-6.el7sat.noarch
rubygem-hammer_cli_foreman_tasks-0.0.3.4-1.el7sat.noarch
foreman-proxy-1.7.2.5-1.el7sat.noarch
ibm-x3755-02.ovirt.rhts.eng.bos.redhat.com-foreman-client-1.0-1.noarch
ibm-x3755-02.ovirt.rhts.eng.bos.redhat.com-foreman-proxy-client-1.0-1.noarch
foreman-vmware-1.7.2.26-1.el7sat.noarch
rubygem-hammer_cli_foreman_bootdisk-0.1.2.7-1.el7sat.noarch
foreman-1.7.2.26-1.el7sat.noarch
ruby193-rubygem-foreman_hooks-0.3.7-2.el7sat.noarch
foreman-debug-1.7.2.26-1.el7sat.noarch

steps:
1.list of content view versions for a content view with over 50 content views

Comment 8 Bryan Kearney 2015-08-11 13:24:13 UTC
This bug is slated to be released with Satellite 6.1.

Comment 9 Bryan Kearney 2015-08-12 13:59:06 UTC
This bug was fixed in version 6.1.1 of Satellite which was released on 12 August, 2015.


Note You need to log in before you can comment on or make changes to this bug.