On Fri, Apr 10, 2015 at 14:58:53 +0200, Ján Tomko wrote:
Create four smaller blocks of texts instead of one large one.
Smaller blocks are user-friendlier and could potentially
make one's life happier.
I'd drop the last sentence before pushing.
---
docs/formatsnapshot.html.in | 41 +++++++++++++++++++++++++----------------
1 file changed, 25 insertions(+), 16 deletions(-)
diff --git a/docs/formatsnapshot.html.in b/docs/formatsnapshot.html.in
index 4f7b7b2..c3ab516 100644
--- a/docs/formatsnapshot.html.in
+++ b/docs/formatsnapshot.html.in
@@ -156,22 +156,31 @@
require that if specified, the snapshot mode must not
override any snapshot mode attached to the corresponding
domain disk, while others like qemu allow this field to
- override the domain default. If the snapshot mode is
- external (whether specified or inherited), then there is
- an optional sub-element <code>source</code>, with an
- attribute <code>file</code> giving the name, and an
- optional sub-element <code>driver</code>, with an
- attribute <code>type</code> giving the driver type (such
- as qcow2), of the new file created by the external
- snapshot of the new file. If <code>source</code> is not
- given and the disk is backed by a local image file (not
- a block device or remote storage), a file name is
- generated that consists of the existing file name
- with anything after the trailing dot replaced by the
- snapshot name. Remember that with external
- snapshots, the original file name becomes the read-only
- snapshot, and the new file name contains the read-write
- delta of all disk changes since the snapshot.
+ override the domain default.
+
+ <dl>
+ <dt><code>source</code></dt>
+ <dd>If the snapshot mode is external (whether specified
+ or inherited), then there is an optional sub-element
+ <code>source</code>, with an attribute
<code>file</code>
+ giving the name of the new file.
+ If <code>source</code> is not
+ given and the disk is backed by a local image file (not
+ a block device or remote storage), a file name is
+ generated that consists of the existing file name
+ with anything after the trailing dot replaced by the
+ snapshot name. Remember that with external
+ snapshots, the original file name becomes the read-only
+ snapshot, and the new file name contains the read-write
+ delta of all disk changes since the snapshot.
Hmmm, I should fix this one later. We now support a full disk source
subXML here along with the "file" attribute. I somehow forgot to
document that :/
+ </dd>
+ <dt><code>driver</code></dt>
+ <dd>An optional sub-element <code>driver</code>,
+ with an attribute <code>type</code> giving the driver type
(such
+ as qcow2), of the new file created by the external
+ snapshot of the new file.
+ </dd>
+ </dl>
<span class="since">Since 1.2.2</span> the
<code>disk</code> element
supports an optional attribute <code>type</code> if the
ACK
Peter