]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Code Review / openstack-build / cinder-build.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: c6b8fae) | patch
LVM: Pass volume size in MiB to copy_volume() during volume migration
authorMitsuhiro Tanino <mitsuhiro.tanino@hds.com>
Wed, 13 May 2015 15:59:13 +0000 (11:59 -0400)
committerMitsuhiro Tanino <mitsuhiro.tanino@hds.com>
Wed, 13 May 2015 20:35:41 +0000 (16:35 -0400)
commita5da4c353a1df17341c1e745796e00a4ec1afd21
treed0ea36f93980ead8dfacc14165976af5e6590513tree | snapshot
parentc6b8faec406b72337b2d1f901be2fa508cbdcfd6commit | diff
LVM: Pass volume size in MiB to copy_volume() during volume migration

Currently migrate_volume() in lvm.py does not pass volume size in
MiB but passes volume size in GiB to copy_volume(). The size argument
of copy_volume() requires MiB value. As a result, if the volume size
is 1GiB, copy_volume() copies only 1MiB head of 1GiB volume to the
destination volume, and then the volume data is corrupted.
This patch fixes to pass volume size in MiB.

Closes-bug: #1454835
Change-Id: I8e06195dc3625ef07bc9858d844541e37a231b73
cinder/tests/unit/test_volume.py diff | blob | history
cinder/volume/drivers/lvm.py diff | blob | history