instance_usage_audit fail because using admin context to talk to glance

Bug #997833 reported by Chris Behrens
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
High
Chris Behrens

Bug Description

https://review.openstack.org/#/c/7309
commit 53adfd289bf7226f1c6f59c17608e42f3083f130

Added pulling of image metadata when sending usage notifications. However, instance_usage_audit only has an admin context... so it'll fail.

We should probably keep track of the image metadata along with the instance, if we need to use it for notifications. The new system_metadata table would be perfect for this!

Chris Behrens (cbehrens)
Changed in nova:
importance: Undecided → High
status: New → Triaged
assignee: nobody → Chris Behrens (cbehrens)
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/7332

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

Reviewed: https://review.openstack.org/7332
Committed: http://github.com/openstack/nova/commit/7d390243c4edc8991d0590273ef48e7d793e6113
Submitter: Jenkins
Branch: master

commit 7d390243c4edc8991d0590273ef48e7d793e6113
Author: Chris Behrens <email address hidden>
Date: Fri May 11 00:10:00 2012 +0000

    Store image properties with instance system_metadata

    In implementing adding 'image_meta' to instance notifications,
    this was committed:

    https://review.openstack.org/#/c/7309
    commit 53adfd289bf7226f1c6f59c17608e42f3083f130

    However, it's buggy because an instance could have been deleted after we
    built from it... and bin/instance-usage-audit uses an admin context with
    which we cannot query glance.

    This stores image properties with instances in nova as system_metadata
    and notifications will use that data.

    Fixes bug 997833

    Change-Id: I50575969b5cb28adaae9a713e749dc486772c417

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