]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Detach volume on device validation failure
authorGorka Eguileor <geguileo@redhat.com>
Fri, 2 Oct 2015 13:08:31 +0000 (15:08 +0200)
committerGorka Eguileor <geguileo@redhat.com>
Mon, 5 Oct 2015 13:39:32 +0000 (15:39 +0200)
commitc5323c6d6c9ec6628cd38e21f11ba45bfeb7867f
treedbb6f631511d80ecfa39b993cad0670c6946f278
parenta992336d07117a8d6d4be3f8bfa9a2fb167dc700
Detach volume on device validation failure

If device validation fails when attaching a volume for some driver
operation (copy_volume_data, copy_image_to_volume, copy_volume_to_image)
we may end up with an attached volume that we don't cleanup.

This patch tries to detach the volume if we fail when validating the
device after we have attached the volume.

This happens for example on multipath when we have properly detected the
paths but they are all in a failed state when we try to read from the
device on validation.

Closes-Bug: #1502138
Change-Id: I73be4206930eba7da064e22d86ff2136191acb0b
cinder/tests/unit/test_volume.py
cinder/volume/driver.py