Failed stats update doesn't include driver name

Bug #1259279 reported by Walt Boring
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Walt Boring

Bug Description

When a driver fails to initialize at startup, the get_volume_stats() will always fail. The problem is that the log file doesn't include the driver name in the update.

"WARNING cinder.volume.manager [-] Unable to update stats, driver is uninitialized"

Tags: driver
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

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

Reviewed: https://review.openstack.org/60924
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=a83506d170e90b111b7e63fab9f585883971f316
Submitter: Jenkins
Branch: master

commit a83506d170e90b111b7e63fab9f585883971f316
Author: Walter A. Boring IV <email address hidden>
Date: Mon Dec 9 11:58:40 2013 -0800

    Add the driver name to get stats log output

    When the manager call tries to update the driver
    stats, it checks to ensure the driver is initialized.
    When the driver hasn't been initialized, it logs
    a warning without the driver name. This is confusing
    for multiple backends. This patch adds the driver
    name, version and the config group name, to the log output,
    so the admin can see which driver is failing.

    Change-Id: I82e8aa969e7baa55db9c8dddc1c08db2a1a41091
    Closes-Bug: #1259279

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: icehouse-2 → 2014.1
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.