]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Clear migration_status from a destination volume if migration fails
authorMitsuhiro Tanino <mitsuhiro.tanino@hds.com>
Mon, 9 Feb 2015 22:18:31 +0000 (17:18 -0500)
committerMitsuhiro Tanino <mitsuhiro.tanino@hds.com>
Fri, 27 Feb 2015 21:39:52 +0000 (16:39 -0500)
commitd96c40c0ab85090a83cb532090cc879c27d630d5
tree3d7bd435762d8cde8b434898eb5b2ec9026a89b7
parent0451354f25432cc7fea1af09650f293fb2c973c0
Clear migration_status from a destination volume if migration fails

When some error happens during volume migration, cinder-volume
keeps remaining a destination volume because the volume copy may be
in the completing phase and a source volume may be deleted already.

In this case, the migration_status of the destination volume in
data base should be cleared. As a result, the volume can be handled
properly even if volume migration is failed. If the migration is
in the migrating phase, the volume should be deleted and an entry
of database is also destroyed because the source volume is remaining.

Also there is an another problem that a source volume is remaining
as migrating status if create_export raises exception. In this case,
source volume can't be deleted even if it will be unnecessary.

This change fixes these two problems.

Closes-Bug: #1403916
Change-Id: Ie4199bc1e804e5cbbb793f448a99487f9823f1a9
cinder/tests/test_volume.py
cinder/volume/manager.py