[libvirt] [PATCH v2 8/8] virsh: Extend virsh dominfo to display if managed state exists

Osier Yang jyang at redhat.com
Tue Jul 19 06:30:58 UTC 2011


于 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




More information about the libvir-list mailing list