Comment 12 for bug 1008866

I understand that we want a better soution, but I think your patch is too large to backport into stable/essex and therefore should be done against the new cinder project.

The simple solution is appropriate for an essex backport. The more complex solution can be done in cinder, as it still requires some cleanup.

For example, the attach to snapshot code is essentially the same as attach to volume code and should be put into a little library of some sort. Also, I couldn't tell from the patch if it is using discovery by default, but if so it should be storing provider_location in the snapshots as well and connecting via an initialize_connection call in the same way it does for volumes.

In other words, this is a large set of changes that should be planned coherently in the cinder project.