]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Remove dm_setup(remove) call in volume_delete.
authorJohn Griffith <john.griffith@solidfire.com>
Tue, 6 Nov 2012 18:08:58 +0000 (11:08 -0700)
committerJohn Griffith <john.griffith@solidfire.com>
Tue, 6 Nov 2012 18:12:18 +0000 (11:12 -0700)
commitc5f3a32178c09f03d617998d7b6d0559ea9e0ca3
tree808181761125dd7f34fd1609db0657b1f38b2aa9
parent84c9dd1ea899796793e9c9c79b0366b71e4a285a
Remove dm_setup(remove) call in volume_delete.

While it makes sense to perform the dm_setup(remove) before
performing the lvm remove, it seems to cause some buffer IO
errors as evident by kern.log.

I'm also a bit suspicous in that it seems the delete timeouts
we see seems to have increased in frequency with this addition.
I propose this step is removed until we completely understand the
impact of the kernel error on Precise kernels.

Fixes bug: #1075675

Change-Id: I7ae4decea42d8f070d774d3e6376686b3e5b4f83
cinder/volume/driver.py