Bug 1020489 - [doc] RHEV-only features documented in the RHEL guides
Summary: [doc] RHEV-only features documented in the RHEL guides
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: doc-Virtualization_Administration_Guide
Version: 6.5
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: ---
Assignee: Laura Novich
QA Contact: ecs-bugs
URL:
Whiteboard:
Depends On:
Blocks: 1035631
TreeView+ depends on / blocked
 
Reported: 2013-10-17 18:40 UTC by Ademar Reis
Modified: 2016-01-27 01:57 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1035631 (view as bug list)
Environment:
Last Closed: 2014-10-16 01:01:10 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description Ademar Reis 2013-10-17 18:40:45 UTC
Some features which are -rhev only are documented in our guides, which are targeted at RHEL users:

virsh snapshot-create --live
  --> this is live-snapshot, a RHEV-only feature

virsh migrate --live --copy-storage-{all,inc}
  --> this is live storage migration, also a RHEV-only feature

There may be other RHEV features documented there, I didn't perform an extensive review.

Some examples:

  Virtualization_Administration_Guide/chap-Virtualization_Administration_Guide-KVM_live_migration.html

  Virtualization_Administration_Guide/virsh-migration-arguements.html

Comment 7 Ademar Reis 2014-10-01 13:24:53 UTC
Reviewing this part:

https://documentation-devel.engineering.redhat.com/site/documentation/en-US/Red_Hat_Enterprise_Linux/6-Beta/html/Virtualization_Administration_Guide/sub-sect-Managing_guest_virtual_machines_with_virsh-Managing_snapshots.html

I see that --live was removed (which is good), but I think that --quiesce and a few other parts of the documentation there don't make sense anymore considering that live-snapshots are not supported. But I would leave this for someone from libvirt to review and suggest editions.

The rest looks good to me.

Comment 8 Ademar Reis 2014-10-01 13:25:46 UTC
(In reply to Ademar Reis from comment #7)
> Reviewing this part:
> 
> https://documentation-devel.engineering.redhat.com/site/documentation/en-US/
> Red_Hat_Enterprise_Linux/6-Beta/html/Virtualization_Administration_Guide/sub-
> sect-Managing_guest_virtual_machines_with_virsh-Managing_snapshots.html
> 
> I see that --live was removed (which is good), but I think that --quiesce
> and a few other parts of the documentation there don't make sense anymore
> considering that live-snapshots are not supported. But I would leave this
> for someone from libvirt to review and suggest editions.
> 
> The rest looks good to me.

Correction: this part needs to be reviewed as well (same reason as above):

https://documentation-devel.engineering.redhat.com/site/documentation/en-US/Red_Hat_Enterprise_Linux/6-Beta/html/Virtualization_Administration_Guide/sect-ext-snapshot-workaround.html

Comment 11 Eric Blake 2014-10-08 19:53:51 UTC
(In reply to Ademar Reis from comment #7)
> Reviewing this part:
> 
> https://documentation-devel.engineering.redhat.com/site/documentation/en-US/
> Red_Hat_Enterprise_Linux/6-Beta/html/Virtualization_Administration_Guide/sub-
> sect-Managing_guest_virtual_machines_with_virsh-Managing_snapshots.html
> 
> I see that --live was removed (which is good), but I think that --quiesce
> and a few other parts of the documentation there don't make sense anymore
> considering that live-snapshots are not supported. But I would leave this
> for someone from libvirt to review and suggest editions.
> 
> The rest looks good to me.

You are correct that --quiesce doesn't work if live disk snapshots aren't supported.

I just looked at the sections Ademar pointed out, and have the following additional comments:

#snapshot-create <domain> <xmlfile> [--redefine [--current] [--no-metadata] [--halt] [--disk-only] [--reuse-external] [--quiesce] [--atomic]

Missing a closing ] on --redefine.

I guess that is output that matches what libvirt outputs (because I recall that we fixed the missing ] at some point), modified to remove --live.  But it is more than just --live that requires qemu help - even --disk-only is a no-op for an offline domain, but can't work for a running domain.  Libvirt isn't going to remove the options; so about the only sane thing you can do is document ALL the options that libvirt provides, along with a caveat that the options are known to cause a graceful error message when driving a qemu that does not support the feature.

--halt, --disk-only, --quiesce, --atomic, and --live are all flags that really only matter when taking a snapshot of a running domain.

"--disk-only - causes the rest of the fields to be ignored, and automatically filled in by libvirt." - not quite.  Really, it is a flag that says that for a running domain, only disk state needs to be saved; it is redundant with snapshot XML that uses   <memory snapshot='no'/> (<memory was added to the XML after the flag had first been implemented).

All offline snapshots are disk-only, and because the guest is offline at the time, there is no need to do fsck or journal replay to get the disk image consistent again (assuming the guest was put offline via normal shutdown).

In section 22, the --memspec option of snapshot-create-as is applicable only to running domain snapshots.

Comment 14 Dayle Parker 2014-10-16 01:01:10 UTC
This change is now available in the Red Hat Enterprise Linux 6.6 documentation at https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/.


Note You need to log in before you can comment on or make changes to this bug.