Instance usage audit log extension miscounts hosts

Bug #1030106 reported by Monsyne Dragon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Undecided
Monsyne Dragon

Bug Description

In the extension that reports the status of the instance usage audit task in nova, hosts that are disabled are not included in the count. this is incorrect. Disabled hosts can still have instances on them and are generally still running if they are not deleted, so they will (and should) run the task.

This results in the log reporting things like "100 of 95 hosts done. 0 errors" on success, which may confuse automated systems using this for monitoring.

Monsyne Dragon (mdragon)
Changed in nova:
assignee: nobody → Monsyne Dragon (mdragon)
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/10427

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

Reviewed: https://review.openstack.org/10427
Committed: http://github.com/openstack/nova/commit/aad7743f42e559aefde37bb44622986df4e2ae98
Submitter: Jenkins
Branch: master

commit aad7743f42e559aefde37bb44622986df4e2ae98
Author: Monsyne Dragon <email address hidden>
Date: Fri Jul 27 19:51:32 2012 +0000

    Correct host count in instance_usage_audit_log extension.

    This fixes bug 1030106. Basically we were not counting disabled hosts
    in the total host count. Also I have refactored the get_audit_task_logs
    method, it's really part of the extension (only the extension uses it),
    so I've moved it out of nova.compute.utils into the extension itself.

    (the tests that exercise it already lived in the extension's tests anyway.)

    Change-Id: Iaf42d887b824ba0cbf6ab0ed143a4c01b37b0cb1

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