[Engine-devel] Floating Disks implementation in REST-API

Geert Jansen gjansen at redhat.com
Wed Apr 11 14:29:31 UTC 2012



On 04/11/2012 02:53 PM, Itamar Heim wrote:
>>> Detach a disk from VM (becomes floating):
>>> DELETE api/vms/{vm:id}/disks/{disk:id}
>>>
>>> Delete a disk ('real' delete)"
>>> DELETE api/disks/{disk:id}
>>
>> Assuming this also works when the disk is attached to a VM then the
>> above seems to me like the simplest and clearest path. i.e.
>> DELETE in VM context detaches
>
> this would break backward compatibility of the API, as today this
> deletes the disk as well.
> geert - thoughts on this?

Yes, the way this is proposed by Ayal would be incompatible. The default 
semantics for a DELETE in a VM context has to be delete, not detach. But 
I don't see that as a problem. We can pass in a 
<detach>true|false</detach> parameter.

Arguably this also more closely aligns with how people will use the API. 
I expect that normally 99% of disks in an environment will be 
non-floating. Therefore a DELETE even in VM context should by default 
really delete the disk, instead of creating a floating disk.

Regards,
Geert



More information about the Engine-devel mailing list