]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Downgrade target create failure mesg to warning
authorJohn Griffith <john.griffith@solidfire.com>
Wed, 23 Oct 2013 20:00:35 +0000 (20:00 +0000)
committerJohn Griffith <john.griffith@solidfire.com>
Wed, 23 Oct 2013 21:10:54 +0000 (15:10 -0600)
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

cinder/brick/iscsi/iscsi.py

index 098ce56248d71061e0cec5ee9fa3acf1c35f8aae..7ea8fd83e6b8e9868dbb5b761ac43dff21098bdb 100644 (file)
@@ -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)