I would like to move forward with that, but changing an existing API makes that 
difficult.

I would suggest to add a second API entry point instead:

__PACKAGE__->register_method({
    name => 'external_migrate_vm',
    path => '{vmid}/external_migrate',
    method => 'POST',
...

Feel free to choose a better name ;-) We can the mark this API as 
unstable/experimental, and modify
the parameters/types. IMHO most existing parameters does not really makes sense 
with external migration.
I guess it is still possible to factor out most common code to avoid code 
duplication. 
 
What do you think?

_______________________________________________
pve-devel mailing list
[email protected]
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to