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 __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: [email protected]?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
