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

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

 



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@xxxxxxxxxx    +1-801-349-2682
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]