]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
make delete recovery messages debug level
authorSean Dague <sean.dague@samsung.com>
Tue, 17 Dec 2013 12:29:46 +0000 (07:29 -0500)
committerSean Dague <sean.dague@samsung.com>
Tue, 17 Dec 2013 12:29:46 +0000 (07:29 -0500)
lvm.delete has recovery logic within it to do a second attempt
if it should fail the first time. It is currently however logging
those second attempts at error and warn levels. However, as this
is normal flow these should be debug level instead.

This is currently causing a few resets in the gate because this
is a non whitelisted error.

Change-Id: Ie684331d9d97c089ec23569d06981e90eb7f6a67
Closes-Bug: #1261731

cinder/brick/local_dev/lvm.py

index 11d89e07d53bc44badf03ad8dddee62729cf427f..8772d1c1c660aa1b2baad0a8973e27e95711ed36 100644 (file)
@@ -574,9 +574,9 @@ class LVM(executor.Executor):
             mesg = (_('Error reported running lvremove: CMD: %(command)s, '
                     'RESPONSE: %(response)s') %
                     {'command': err.cmd, 'response': err.stderr})
-            LOG.error(mesg)
+            LOG.debug(mesg)
 
-            LOG.warning(_('Attempting udev settle and retry of lvremove...'))
+            LOG.debug(_('Attempting udev settle and retry of lvremove...'))
             self._execute('udevadm', 'settle',
                           root_helper=self._root_helper,
                           run_as_root=True)