nova api paginating but dashboard is not

Bug #1046915 reported by Hendrik Volkmer
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Medium
David Lyle

Bug Description

The nova API (e.g. server list) paginates but the dashboard list does not.

e.g. When I start 2000 VMs in one tenant it only shows 1000 VMs in the list.

Steps to reproduce:
1) set quota on tenant to allow more than 1000 VMs (and enough VCPU, RAM, disk resources)
2) start 1001 VMs
3) wait for VMs to build
4) see only first 1000 VMs (both on the Tenant view at /nova/instances_and_volumes/ as well as on the admin page at /syspanel/instances/)

Expected results:

a) All VMs are shown
- or -
b) The dashboard itself uses pagination and shows about 100 VMs per page

description: updated
Changed in horizon:
importance: Undecided → Medium
status: New → Confirmed
Changed in horizon:
milestone: none → grizzly-1
assignee: nobody → Nebula (nebula)
Changed in horizon:
assignee: Nebula (nebula) → nobody
Changed in horizon:
milestone: grizzly-1 → grizzly-2
Changed in horizon:
milestone: grizzly-2 → grizzly-3
Changed in horizon:
milestone: grizzly-3 → none
Revision history for this message
David Lyle (david-lyle) wrote :

I have an implementation of this working in my sandbox, is there any desire to preserve the view all, or just support pagination? The amount of data that can come across especially in the admin case for view all makes the UI virtually unusable. I think pagination only is the best approach, but I could see the use case for all especially given the transient nature of some instances. Thoughts?

Changed in horizon:
assignee: nobody → David Lyle (david-lyle)
Revision history for this message
Julie Pichon (jpichon) wrote :

As far as I can tell, other pages like Images that provide pagination only support pagination without a "View all" option, it seems reasonable to me to do the same for Instances.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Changed in horizon:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/26022
Committed: http://github.com/openstack/horizon/commit/81247fda009e3c27e377b605c5b997279b2f91f6
Submitter: Jenkins
Branch: master

commit 81247fda009e3c27e377b605c5b997279b2f91f6
Author: David Lyle <email address hidden>
Date: Thu Mar 14 08:43:33 2013 -0600

    Adding pagination to the instance views

    Modifying the api.nova.server_list() method to optionally handle
    pagination. The method will also work without pagination to
    support the many other place than the instance views that continue
    to call the method.

    Fixes: bug #1046915

    Change-Id: I8195f1f2d8922e1722743d7a2d627a8645e8b3bd

Changed in horizon:
status: In Progress → Fix Committed
Kieran Spear (kspear)
Changed in horizon:
milestone: none → havana-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/28492
Committed: http://github.com/openstack/horizon/commit/9172a7ba0b150042d286702e1c339f3a6b6c7793
Submitter: Jenkins
Branch: master

commit 9172a7ba0b150042d286702e1c339f3a6b6c7793
Author: Nicolas Simonds <email address hidden>
Date: Tue May 7 18:58:53 2013 -0700

    Make pagination tunable through the settings panel

    This patch set expands on the fine work done in https://review.openstack.org/26022

    It makes the number of items displayed a per-user tunable through
    the "Settings" panel and a session cookie, with the default/max
    values being the values set in django.conf.settings

    Fixes Bug: 1046915
    Change-Id: I38ef4845d46bce62e44209865885aff8eb0bac83

Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: havana-1 → 2013.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.