allowed_instances should check both of instance_type['vcpus'] and instance_type['memory_mb'] are not zero

Bug #883310 reported by Nachi Ueno
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Mandell
OpenStack Quality Assurance
Fix Released
Medium
Mandell

Bug Description

If instance_type['memory_mb'] or instance_type['vcpus'] are zero, this method throws ZeroDivisionError.
https://github.com/openstack/nova/blob/master/nova/quota.py#L100

Nachi Ueno (nati-ueno)
Changed in nova:
assignee: nobody → Nachi Ueno (nati-ueno)
status: New → Confirmed
Changed in openstack-qa:
assignee: nobody → Nachi Ueno (nati-ueno)
status: New → Confirmed
importance: Undecided → Medium
Thierry Carrez (ttx)
Changed in nova:
importance: Undecided → Low
Revision history for this message
Nachi Ueno (nati-ueno) wrote :
tags: added: forward-port-needed
Nachi Ueno (nati-ueno)
Changed in openstack-qa:
assignee: Nachi Ueno (nati-ueno) → nobody
Changed in nova:
assignee: Nachi Ueno (nati-ueno) → nobody
Mandell (mdegerne)
Changed in openstack-qa:
assignee: nobody → Mandell (mdegerne)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

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/3724
Committed: http://github.com/openstack/nova/commit/d9b6338f1c62e17d29057e3e568f0fb3a524a0f3
Submitter: Jenkins
Branch: master

commit d9b6338f1c62e17d29057e3e568f0fb3a524a0f3
Author: Mandell Degerness <email address hidden>
Date: Fri Feb 3 00:01:29 2012 +0000

    Fix for bug 883310

    Don't throw exception when flavor has vcpus=0 or memory_mb=0

    Change-Id: I6f12cf06a569b491f537880afe37f2cf516be2d2

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → essex-4
status: Fix Committed → Fix Released
Mark McLoughlin (markmc)
Changed in nova:
assignee: nobody → Mandell (mdegerne)
Thierry Carrez (ttx)
Changed in nova:
milestone: essex-4 → 2012.1
Nachi Ueno (nati-ueno)
Changed in openstack-qa:
status: In Progress → Fix Released
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.