]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix create_export/remove_export in driver.py
authorAvishay Traeger <avishay@il.ibm.com>
Wed, 26 Feb 2014 09:27:06 +0000 (11:27 +0200)
committerJohn Griffith <john.griffith@solidfire.com>
Sat, 15 Mar 2014 14:12:47 +0000 (14:12 +0000)
commitb868ae707f9ecbe254101e21d9d7ffa0b05b17d1
tree571a73395bf666e45d4c831b9936282707d331d2
parent7010514918f05e6bf804a580a64ace3ff8874921
Fix create_export/remove_export in driver.py

1. There was a call to rpcapi.create_export which does not have a
matching volume manager function, causing volume migration to crash.
This call was not necessary because there is already an
initialize_connection RPC call, which calls the driver's create_export
function. Removed the create_export RPC call and function. Added better
error handling to that code in _attach_volume in driver.py as well.

2. The manager called remove_export from detach_volume, which was not
being called by these functions. I believe it makes more sense to call
it from terminate_connection. Moved it there, and fixed up the
corresponding code in _detach_volume in driver.py.

3. Remove leftovers of export code from create_volume manager flow.

Change-Id: I2b192630ebed54368f151db47b49cbc72601a8d7
Closes-Bug: #1285060
cinder/tests/test_coraid.py
cinder/tests/test_volume.py
cinder/volume/driver.py
cinder/volume/flows/manager/create_volume.py
cinder/volume/manager.py
cinder/volume/rpcapi.py