Re: [PATCH] virsh: Add missing "async" option in virsh-domain.c opts_block_commit.

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

 



On 09/19/2013 07:08 AM, Simone Gotti wrote:
> After commit 8aecd351266a66efa59b7f7be77bf66693d99ce0 it'll detect

Yay - my sanity checking found the first victim!  Boohoo - that victim
was me (the bug has been latent for 1 year now, introduced in ed23b1066).

> that a required option is not defined and it will assert and exit with:
> 
> virsh.c:1364: vshCommandOpt: Assertion `valid->name' failed.
> ---
>  tools/virsh-domain.c | 4 ++++
>  1 file changed, 4 insertions(+)

ACK.  Needs an update to virsh.pod to match, so I squashed this in
(borrowing from blockcopy) and pushed.  Congratulations on your first
libvirt patch.

diff --git c/tools/virsh.pod w/tools/virsh.pod
index 0ae5178..2864f3d 100644
--- c/tools/virsh.pod
+++ w/tools/virsh.pod
@@ -737,7 +737,7 @@ I<domif-setlink>) will accept the MAC address
printed by this command.

 =item B<blockcommit> I<domain> I<path> [I<bandwidth>]
 {[I<base>] | [I<--shallow>]} [I<top>] [I<--delete>]
-[I<--wait> [I<--verbose>] [I<--timeout> B<seconds>]]
+[I<--wait> [I<--verbose>] [I<--timeout> B<seconds>] [I<--async>]]

 Reduce the length of a backing image chain, by committing changes at the
 top of the chain (snapshot or delta files) into backing images.  By
@@ -756,7 +756,10 @@ operation can be checked with B<blockjob>.
However, if I<--wait> is
 specified, then this command will block until the operation completes,
 or cancel the operation if the optional I<timeout> in seconds elapses
 or SIGINT is sent (usually with C<Ctrl-C>).  Using I<--verbose> along
-with I<--wait> will produce periodic status updates.
+with I<--wait> will produce periodic status updates.  If job cancellation
+is triggered, I<--async> will return control to the user as fast as
+possible, otherwise the command may continue to block a little while
+longer until the job is done cleaning up.

 I<path> specifies fully-qualified path of the disk; it corresponds
 to a unique target name (<target dev='name'/>) or source file (<source

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]