From: John Griffith <john.griffith@solidfire.com>
Date: Wed, 23 Oct 2013 20:00:35 +0000 (+0000)
Subject: Downgrade target create failure mesg to warning
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=587fd9021982e3b1fb0c5765df6bad2723701031;p=openstack-build%2Fcinder-build.git

Downgrade target create failure mesg to warning

Since this will occur more often than we would like and
almost always recovers, change the log level to warning
instead of error.

This is to comply with the desire of the QA team to eradicate
error messages from succesful tempest runs.

Change-Id: If1a841bbeb9a36ff7e103a0e9aff67fda66a7f3b
Closes-Bug: #1243840
---

diff --git a/cinder/brick/iscsi/iscsi.py b/cinder/brick/iscsi/iscsi.py
index 098ce5624..7ea8fd83e 100644
--- a/cinder/brick/iscsi/iscsi.py
+++ b/cinder/brick/iscsi/iscsi.py
@@ -207,9 +207,9 @@ class TgtAdm(TargetAdmin):
                                        run_as_root=True)
             LOG.debug("Targets after update: %s" % out)
         except putils.ProcessExecutionError as e:
-            LOG.error(_("Failed to create iscsi target for volume "
+            LOG.warning(_("Failed to create iscsi target for volume "
                         "id:%(vol_id)s: %(e)s")
-                      % {'vol_id': vol_id, 'e': str(e)})
+                        % {'vol_id': vol_id, 'e': str(e)})
 
             #Don't forget to remove the persistent file we created
             os.unlink(volume_path)