]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Fixes brick Nova pep8 violation for lvm.py
authorWalter A. Boring IV <walter.boring@hp.com>
Wed, 4 Sep 2013 18:27:17 +0000 (11:27 -0700)
committerWalter A. Boring IV <walter.boring@hp.com>
Wed, 4 Sep 2013 18:27:17 +0000 (11:27 -0700)
Running pep8 tests in Nova gives a violation
in lvm.py for not importing a module.  This
is a simple fix to import the module itself
instead of only importing a function in the
module.

Fixes bug #1220849

Change-Id: Idc603dcfab18601dd4246c623d34be5933434f84

cinder/brick/local_dev/lvm.py

index b0d4a4d075a3c19834a0428b94bf37edcd19fcfd..c55975fb6b7cb91de4c512a13971c8a0780c0b11 100644 (file)
@@ -22,7 +22,7 @@ LVM class for performing LVM operations.
 import math
 import re
 
-from itertools import izip
+import itertools
 
 from cinder.brick import exception
 from cinder.brick import executor
@@ -169,7 +169,7 @@ class LVM(executor.Executor):
         lv_list = []
         if out is not None:
             volumes = out.split()
-            for vg, name, size in izip(*[iter(volumes)] * 3):
+            for vg, name, size in itertools.izip(*[iter(volumes)] * 3):
                 lv_list.append({"vg": vg, "name": name, "size": size})
 
         return lv_list