[Launch Instance Fix] source step - fix donut chart dynamic update - limit instance count

Bug #1432897 reported by Brian Tully
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Undecided
Brian Tully

Bug Description

In the current version of the Launch Instance Wizard Source Step, the instance count field is not correctly limited based on current instance usage and nova limits (maxTotalInstances). There is a bug whereby the user can enter an instance count of more than what is available, and in addition the donut chart does not update properly when this occurs. We should correct this so that if a user enters a number greater than what is available, it should automatically round down to the available count.

Example. If I have 10 max instances quota, and currently have 6 instances in use, I should only be able to specify 4 as my instance count (which will update the donut chart to 100%). However there is nothing in the code that limits me from entering a higher number. We should add some code that limits the instance count based on # remaining, and automatically rounds down to the highest number of instances that would prevent the user from going over 100%

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/164912

Changed in horizon:
assignee: nobody → Brian Tully (brian-tully)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/164912
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=02acf125597ef48c5e07d80e5f59d189c6fee6ec
Submitter: Jenkins
Branch: master

commit 02acf125597ef48c5e07d80e5f59d189c6fee6ec
Author: Brian Tully <email address hidden>
Date: Mon Mar 16 21:02:41 2015 -0400

    Source step - fix instance count limit and donut chart

    In the current version of the Launch Instance Wizard Source Step,
    the instance count field is not correctly limited based on current
    instance usage and nova limits (maxTotalInstances). There is a bug
    whereby the user can enter an instance count of more than what is
    available (exceeding their quota). In addition, the donut chart
    does not update properly when this occurs. This patch corrects this
    so that if a user enters a number greater than what is available,
    it automatically changes the count to the maximum count that will still
    stay within quota limits.

    Closes-Bug: 1432897

    Change-Id: I24f1309b04f3c045c02b67409c49fc3ee86aea3f

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
milestone: none → kilo-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: kilo-3 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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