Inspector should check the domain state before inspecting nics and disks

Bug #1271454 reported by Liusheng
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
Liusheng

Bug Description

In my devstack environment(compute virt driver is libvirt), there is a shut-off instance. The ceilometer-agent-compute will record the following text in log every time of inspecting nics or disks,and inspect faild:

http://paste.openstack.org/show/61661/

I think the inspector of ceilometer should check the domain state before inspecting nics of disks,and record more conciser log message in log file.

Liusheng (liusheng)
summary: - before inspecting nics and disks,inspector should check the domain state
+ Inspector should check the domain state before inspecting nics and disks
Changed in ceilometer:
assignee: nobody → Liusheng (liusheng)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

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

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

Reviewed: https://review.openstack.org/68882
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=39f13fa706e79f1d56694d1c027b0f808ed5a68e
Submitter: Jenkins
Branch: master

commit 39f13fa706e79f1d56694d1c027b0f808ed5a68e
Author: liu-sheng <email address hidden>
Date: Fri Jan 24 18:03:29 2014 +0800

    check domain state before inspecting nics/disks

    If there are shutoff instances,the ceilometer-inspector vill
    inspect faild,and record the tedious error string in log.
    The inspector of ceilometer should check the domain state before
    inspecting nics and disks,and record more conciser log message in log file.

    Change-Id: Id56bf75b47edf1f3b4ba77f40a48d83186d0afb6
    Closes-bug: #1271454

Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: none → icehouse-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: icehouse-3 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.