]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Merge "Amend unused variables to assist pylint testing"
authorJenkins <jenkins@review.openstack.org>
Wed, 19 Nov 2014 23:41:01 +0000 (23:41 +0000)
committerGerrit Code Review <review@openstack.org>
Wed, 19 Nov 2014 23:41:01 +0000 (23:41 +0000)
12 files changed:
1  2 
cinder/backup/drivers/swift.py
cinder/brick/iscsi/iscsi.py
cinder/brick/local_dev/lvm.py
cinder/brick/remotefs/remotefs.py
cinder/tests/test_backup_swift.py
cinder/tests/test_glusterfs.py
cinder/tests/test_gpfs.py
cinder/volume/drivers/hds/hds.py
cinder/volume/drivers/netapp/iscsi.py
cinder/volume/drivers/netapp/nfs.py
cinder/volume/drivers/san/hp/hp_3par_common.py
cinder/volume/drivers/solidfire.py

Simple merge
Simple merge
index 252fa05d0dd243ef7f6b85244685ab0943256976,205c2db03a4ba75fbfc9507750327e3d42088d60..562fd761a620bc2b607521293ad9e06b10d73161
@@@ -253,12 -249,12 +253,12 @@@ class LVM(executor.Executor)
              cmd.append(vg_name)
  
          lvs_start = time.time()
-         (out, err) = putils.execute(*cmd,
-                                     root_helper=root_helper,
-                                     run_as_root=True)
+         (out, _err) = putils.execute(*cmd,
+                                      root_helper=root_helper,
+                                      run_as_root=True)
          total_time = time.time() - lvs_start
          if total_time > 60:
 -            LOG.warning(_('Took %s seconds to get logical volumes.'),
 +            LOG.warning(_LW('Took %s seconds to get logical volumes.'),
                          total_time)
  
          lv_list = []
              cmd.append(vg_name)
  
          start_vgs = time.time()
-         (out, err) = putils.execute(*cmd,
-                                     root_helper=root_helper,
-                                     run_as_root=True)
+         (out, _err) = putils.execute(*cmd,
+                                      root_helper=root_helper,
+                                      run_as_root=True)
          total_time = time.time() - start_vgs
          if total_time > 60:
 -            LOG.warning(_('Took %s seconds to get volume groups.'), total_time)
 +            LOG.warning(_LW('Took %s seconds to get '
 +                            'volume groups.'), total_time)
  
          vg_list = []
          if out is not None:
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge