于 2011年07月19日 01:37, Eric Blake 写道:
On 07/18/2011 11:34 AM, Eric Blake wrote:
> Here, we print "managed state", but the command that creates that state
> is called "managedsave", and the API we call is
> virDomainHasManagedSaveImage. Not to mention that it surpasses 15
> columns, which makes the output not lined up with all the other rows.
> Also, the "Has" is a bit redundant. I'd prefer to see this output as:
>
> Managed save: yes|no|unknown
>
> ACK with that nit fixed, and this can be pushed independently of the
> rest of the series.
Also, squash in some documentation:
diff --git i/tools/virsh.pod w/tools/virsh.pod
index 5afa1f2..312ddf0 100644
--- i/tools/virsh.pod
+++ w/tools/virsh.pod
@@ -514,6 +514,9 @@ Save and destroy (stop) a running domain, so it
can be restarted from the same
state at a later time. When the virsh B<start> command is next run for
the domain, it will automatically be started from this saved state.
+The B<dominfo> command can be used to query whether a domain currently
+has any managed save state.
+
=item B<managedsave-remove> I<domain-id>
Remove the B<managedsave> state file for a domain, if it exists. This
Pushed with the document, and the nits you mentioned in previous mail
fixed. Thanks
Osier