Comment 1 for bug 1648972

Revision history for this message
John Griffith (john-griffith) wrote :

Some of the base code for this is a bit confusing and there are different ways to do it. I'm unclear on what changes may have impacted you, but looking at your driver I see the problem I think.

Take a look at the base driver copy_image_to_volume call:
    https://github.com/openstack/cinder/blob/master/cinder/volume/driver.py#L785

The lvm driver IIRC is now truly just LVM, when it actually gets instantiated it's going to pick in the iscsi class from driver.py and use that, so you'll then actually end up using the call above in driver.py.

You can also check some of the other drivers that chose to implement the method themeselves. The other option is you shouldn't need to implement this at all, you can just omit it and pick up the method from the base driver.

This is all sort of confusing and needs to be cleaned up a good bit, but check those things out as I think that's where your problem lies. I would be interested in digging in to the changes that exposed this for you though.