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

Re: [libvirt] [PATCH] Document snapshot virsh commands in the man page.



On 04/05/2010 11:37 AM, Chris Lalancette wrote:
> +=item B<snapshot-create> I<domain> I<xmlfile>
> +
> +Create a snapshot for domain I<domain> with the properties specified in
> +I<xmlfile>.  The only properties settable for a domain snapshot are the
> +<name> and <description>; the rest of the fields are ignored, and
> +automatically filled in by libvirt.  If I<xmlfile> is completely ommitted,

s/ommitted/omitted/

> +=item B<revert-to-snapshot> I<domain> I<snapshot>
> +
> +Revert the given domain to the snapshot specified by I<snapshot>.  Be aware
> +that this is a destructive action; any changes in the domain since the
> +snapshot is taken will be lost.  Also note that the state of the domain after

I think it reads better to use s/snapshot is taken/snapshot was taken/,
since the snapshot was created in the past compared to the current state
of the domain that we are discarding.

ACK, with those nits addressed.

We also have some other missing documentation in virsh.pod; for example,
iface-edit and pool-edit are undocumented.

-- 
Eric Blake   eblake redhat com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


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