]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Check volume_backend in retype
authorJohn Griffith <john.griffith8@gmail.com>
Mon, 11 May 2015 19:55:41 +0000 (13:55 -0600)
committerJohn Griffith <john.griffith8@gmail.com>
Thu, 28 May 2015 01:27:42 +0000 (19:27 -0600)
commit03562c362cfa217ec794dd98f197f623c1ffe45d
tree23bafd995febf4ff23909cec237ec6f386d466b0
parentc0f12f28be44a2e05932de1f222698dce221b2b7
Check volume_backend in retype

The retype command will always attempt a call
to the driver.retype method.  In *most* cases this
will hit the default impl which returns False because
most drivers don't implement any retype (most, a few do).

The problem is that the drivers that do implement this in
most cases will iterate through the settings and just make
the changes that are valid and ignore the rest, and then
return True. I think this is "ok" for the drivers to do,
drivers should be allowed to be somewhat dumb WRT Cinder
state management and placement info.  If we give them an
invalid command (which we're doing here) then it's on us
higher up the chain IMO.

The result is that for example if you're trying to retype
from backend-a to backend-b and backend-a implements retype
it can return True telling the manager that the volume was
succesfully retyped, even when it wasn't.

There's a lot of confusion around this bug, YES the
filter scheduler is used to determine if the retype is
valid and to what host.  That's not the issue, the issue
is that regardless of the source and destination host settings
that are provided from the filter-scheduler, we always make the
call to the driver, introducing the opportunity for a false
success status being reported back.

This patch adds a very simple check between the source and
destination host settings as provided by the scheduler and in
the case that the two are "different"(not including pool designations)
we skip calling the driver.retype method altogether and fall through
to the migrate process.

This introduces a new hosts_are_equivalent method in
cinder.volume.utils

Conflicts:
cinder/volume/utils.py

Closes-Bug: #1452823
(cherry picked from commit bbb83aff023c90cfc186459603320ad3c622a40b)
Change-Id: Idfd7dfa2284fcea66cf23c4273efda61ee8f7eba
cinder/tests/test_volume_utils.py
cinder/volume/manager.py
cinder/volume/utils.py