Comment 11 for bug 1298350

Ramashri, good finding! Also I've looked at the history of VolumeDetachTask and noticed that it uses Nova client actions to work with volumes. I think it's designed this way because in that time of DetachTask develompent Cinder client didn't have all needed rest calls to work with. Now Cinder and its client contains all needed functions, and probably Cinder::Volume and Cinder::VolumeAttachment should be rewritten to use it. As a proof I can show an example that in Sahara provisioning code works fine in the similar case just having this code:
https://github.com/openstack/sahara/blob/master/sahara/service/volumes.py#L157