From: Eric Harney <eharney@redhat.com>
Date: Wed, 12 Jun 2013 21:20:52 +0000 (-0400)
Subject: Elevate volume/snap "is busy" log message for volume/snap_delete
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=99f08868b8c5227622b3ac84c8863a25a161953d;p=openstack-build%2Fcinder-build.git

Elevate volume/snap "is busy" log message for volume/snap_delete

If deletion fails because the volume is busy, log at "error" level
rather than "debug" to ensure that the error is logged and visible
to a user.

Change-Id: I04031ee6b1f6647a126baa8a29c15a00b84b27b1
---

diff --git a/cinder/volume/manager.py b/cinder/volume/manager.py
index 97bcd958e..e986b7cd0 100644
--- a/cinder/volume/manager.py
+++ b/cinder/volume/manager.py
@@ -424,7 +424,8 @@ class VolumeManager(manager.SchedulerDependentManager):
             LOG.debug(_("volume %s: deleting"), volume_ref['name'])
             self.driver.delete_volume(volume_ref)
         except exception.VolumeIsBusy:
-            LOG.debug(_("volume %s: volume is busy"), volume_ref['name'])
+            LOG.error(_("Cannot delete volume %s: volume is busy"),
+                      volume_ref['name'])
             self.driver.ensure_export(context, volume_ref)
             self.db.volume_update(context, volume_ref['id'],
                                   {'status': 'available'})
@@ -507,7 +508,8 @@ class VolumeManager(manager.SchedulerDependentManager):
             LOG.debug(_("snapshot %s: deleting"), snapshot_ref['name'])
             self.driver.delete_snapshot(snapshot_ref)
         except exception.SnapshotIsBusy:
-            LOG.debug(_("snapshot %s: snapshot is busy"), snapshot_ref['name'])
+            LOG.error(_("Cannot delete snapshot %s: snapshot is busy"),
+                      snapshot_ref['name'])
             self.db.snapshot_update(context,
                                     snapshot_ref['id'],
                                     {'status': 'available'})