]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Brick LVM: Rename get_all_volumes, further optimize
authorEric Harney <eharney@redhat.com>
Tue, 28 Oct 2014 21:03:42 +0000 (17:03 -0400)
committerEric Harney <eharney@redhat.com>
Wed, 29 Oct 2014 13:53:29 +0000 (09:53 -0400)
commit2cb519dbd20de68b1c182e772621098527e97e17
treee57133f56c7c873ff2b83af3ec3a6e82b3627f7e
parentdcda67053df5dc0240c743537dc1b9c4a3231b61
Brick LVM: Rename get_all_volumes, further optimize

This method can process all LVs, those in a VG, or an
individual LV.  Rename it to be more descriptive.

Optimize update_volume_group_info check for thin pool.

Remove volume.utils.get_all_volumes, which is unused.

Remove fake_lvm.get_all_volumes and
fake_lvm.get_all_volume_groups which don't appear to
have correct signatures/returns and are unused.

Change-Id: I412c62230517cbfc08cc0076aaeea45224e6b50e
cinder/brick/local_dev/lvm.py
cinder/tests/brick/fake_lvm.py
cinder/volume/utils.py