Invalid validation for unlimited values when updating user quota

Bug #1292494 reported by Gastón Severina
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Gastón Severina

Bug Description

There is a possible scenario that is not correctly handled when validating quota limit. If the remaining quota is unlimited (-1) and new quota value is unlimited (-1) then both values are summed resulting in a -2 value that then causes the following error: "Quota limit must be greater than 0".

Changed in nova:
assignee: nobody → Gastón Severina (gseverina)
Revision history for this message
Christopher Yeoh (cyeoh-0) wrote :

It looks like we have a general problem with estimating what the maximum quota can be if it is currently unlimited. Needs further investigation for more than just this specific case (I think we'll often have off by one errors).

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

Reviewed: https://review.openstack.org/80615
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=362e998e89a1f1d9dce3379c4610b205ebeec7c0
Submitter: Jenkins
Branch: master

commit 362e998e89a1f1d9dce3379c4610b205ebeec7c0
Author: gseverina <email address hidden>
Date: Fri Mar 14 13:06:45 2014 -0300

    Handling unlimited values when updating quota

    Changes:
    - Adding a general approach to handle unlimited values when performing
    calculations with quota values.
    - New test that fails without the fix.
    - Updated test cases to support the fix.

    Change-Id: I762a8e67261c0e0286307832cd13ad12e6d96504
    Closes-Bug: #1292494

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → juno-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: juno-1 → 2014.2
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.