Failed stats update doesn't include driver name

Bug #1259279 reported by Walt Boring on 2013-12-09
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
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"

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

Changed in cinder:
status: New → In Progress

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) on 2014-01-22
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2014-04-17
Changed in cinder:
milestone: icehouse-2 → 2014.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers