Open Stack

On 10:58 Jun 29, Sheng Bo Hou wrote:
> Hi everyone,
> 
> For folks who are working on Dell, Storpool and Infortrend drivers:
> I have got a new patch for https://review.openstack.org/#/c/180873. There 
> is a change about how to implement the method update_migrated_volume for 
> each driver.
> The code needs to return the final values for the key _name_id and 
> provider_location. I have changed accordingly in your driver, but I am no 
> 100% sure of the precision, so I need your reviews on the changes about 
> your driver. If there is any comment, tell me how to change it. You can 
> take the implementation for Storwize as a reference.
> First, check if the rename works the same for both the 'available' or 
> 'in-use' volumes. It is possible to handle differently.
> Then check if the rename is successful, it may return different _name_id 
> and provider_location values.
> There is an explanation about the update_migrated_volume in 
> https://review.openstack.org/#/c/180873/67/cinder/volume/driver.py

Lets mention this in the next Cinder meeting and if you could, reach out to the
people in the git logs for those volume drivers.

-- 
Mike Perez