]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Remove locals() from iser
authorWalter A. Boring IV <walter.boring@hp.com>
Tue, 6 Aug 2013 21:43:35 +0000 (14:43 -0700)
committerWalter A. Boring IV <walter.boring@hp.com>
Tue, 6 Aug 2013 21:43:35 +0000 (14:43 -0700)
This patch removes the locals() call
to fix a flake8 violation.

Fixes Bug #1209006

Change-Id: Ida4b498544a2c28ffb4912d402579f8bb7372309

cinder/brick/iser/iser.py

index b82c2d49d744d680eb9b42e58ad82fed72b95ce3..cafb2ac951e1acd980f1fd1885a4d02ecfa93ae6 100755 (executable)
@@ -168,7 +168,8 @@ class TgtAdm(TargetAdmin):
         if tid is None:
             LOG.error(_("Failed to create iser target for volume "
                         "id:%(vol_id)s. Please ensure your tgtd config file "
-                        "contains 'include %(volumes_dir)s/*'") % locals())
+                        "contains 'include %(volumes_dir)s/*'") %
+                      {'vol_id': vol_id, 'volumes_dir': volumes_dir})
             raise exception.NotFound()
 
         if old_persist_file is not None and os.path.exists(old_persist_file):