Re: [PATCHv5 05/23] blockjob: add 'blockcopy' to virsh

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 16, 2012 at 23:05:56 -0600, Eric Blake wrote:
> Rather than further overloading 'blockpull', I decided to create a
> new virsh command to expose the new flags of virDomainBlockRebase.
> 
> Blocking until the command completes naturally is pointless, since
> the block copy job is intended to run indefinitely.  Instead, I
> made the command support three --wait modes: by default, it runs until
> mirroring is started; with --pivot, it pivots as soon as mirroring
> is started; and with --finish, it aborts (for a clean copy) as
> soon as mirroring is started.
> 
> * tools/virsh.c (VSH_CMD_BLOCK_JOB_COPY): New mode.
> (blockJobImpl): Support new flags.
> (cmdBlockCopy): New command.
> (cmdBlockJob): Support new job info, new abort flag.
> * tools/virsh.pod (blockcopy, blockjob): Document the new command
> and flags.
> ---
> 
> was 6/18 in v4
> v5: add --wait flag, consistent capitalization
> 
>  tools/virsh.c   |  212 +++++++++++++++++++++++++++++++++++++++++++++++++++----
>  tools/virsh.pod |   49 ++++++++++++-
>  2 files changed, 243 insertions(+), 18 deletions(-)

OK, looks good but my comment from patch 4/23 applies here as well.

Jirka

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]