simple_tenant_usage extension should not rely on looking up flavors

Bug #1276731 reported by Dan Smith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
jichenjc

Bug Description

The simple_tenant_usage extension gets the flavor data from the instance and then looks up the flavor from the database to return usage information. Since we now store all of the flavor data in the instance itself, we should use that information instead of what the flavor currently says is right. This both (a) makes it more accurate and (b) avoids us failing to return usage info if a flavor disappears.

Tags: api
Matt Riedemann (mriedem)
tags: added: api
Changed in nova:
status: New → Confirmed
jichenjc (jichenjc)
Changed in nova:
assignee: nobody → jichencom (jichenjc)
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/76230

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/76230
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=c9a020f121a85ff88b890e964dc8c1f6d0ff5843
Submitter: Jenkins
Branch: master

commit c9a020f121a85ff88b890e964dc8c1f6d0ff5843
Author: jichenjc <email address hidden>
Date: Tue Feb 25 20:00:09 2014 +0800

    Use instance data instead of flavor in simple_tenant_usage extension

    The simple_tenant_usage extension gets the flavor data from the instance
    and then looks up the flavor from the database to return usage information.
    Since we now store all of the flavor data in the instance itself,
    we should use that information instead of what the flavor currently
    says is right. This both (a) makes it more accurate and (b)
    avoids us failing to return usage info if a flavor disappears.

    Change-Id: I1472d78e8e920013a1db5c76e04715fa04b696c2
    Closes-Bug: #1276731

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.