Re: [Qemu-devel] [PATCH v5 3/6] add backup related monitor commands

[Dietmar Maurer]

> > You can easily extend the API to allow multiple backups (In a fully
> > backwards compatible way). So there is no need to change that now.
> 
> I disagree.
> 
> You propose something like:
> 
> 1.  -> { "execute": "backup",
>          "arguments": { "backup-file": "foo.vma" } }
>     <- { "return": { "deb8e5da-5b69-46d1-86c6-1b715a22809f" } }
> 
>     This *deletes* any prior BackupStatus.  Clearly inappropriate with
>     multiple backups.  How exactly do you propose to extend it in a
>     backwards compatible way?

It currently deletes the BackupStatus. But this is related to the current 
implementation, not the API definition.

> 2.  -> { "execute": "query-backup" }
> 
>     This is specified to return exactly one BackupStatus.
> 
>     How exactly do you propose to extend it for multiple backups?

Add an optional 'uuid' parameter. We can auto-select the job if there is only 
one job running
 
> 3.  -> { "execute": "backup-cancel" }
> 
>     This is specified to cancel "the current execute backup process".
> 
>     This becomes *ill-defined* when multiple backup processes are
>     executing.

Again, we can add an optional 'uuid' parameter.