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

Eric Blake eblake at redhat.com
Mon Apr 5 21:05:18 UTC 2010


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 at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 323 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100405/6e37bebd/attachment-0001.sig>


More information about the libvir-list mailing list