Bug 1343923

Summary: [Docs][VMM] Remove 'Important' notes about manually freezing and thawing filesystems
Product: Red Hat Enterprise Virtualization Manager Reporter: Daniel Erez <derez>
Component: DocumentationAssignee: Byron Gravenorst <bgraveno>
Status: CLOSED CURRENTRELEASE QA Contact: Megan Lewis <melewis>
Severity: unspecified Docs Contact:
Priority: high    
Version: 3.6.0CC: amureini, derez, lbopf, lsurette, rbalakri, srevivo, ykaul, ylavi, zdover
Target Milestone: ovirt-3.6.10Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-05-03 01:55:30 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Docs RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Daniel Erez 2016-06-08 10:33:06 UTC 
Description of problem:

Virtual Machine Management Guide:
6.8.1. Creating a Snapshot of a Virtual Machine

1)
The 'IMPORTANT' note is wrong. I.e. no need to manually freeze and thaw the guest filesystem when using the REST API. It is done automatically by the engine. These actions where added just as a safety measure to mitigate issues with live snapshotting a VM with Cinder disks. E.g. it should be invoked manually if the VM stayed on freeze state after taking a live snapshot (due to a failure).

2)
Procedure 6.10. Creating a Snapshot of a Virtual Machine

The note is wrong. 'thaw' should be manually executed only in case of a failure (only if the VM stayed on freeze state).
Comment 1 Yaniv Lavi 2016-06-14 11:53:30 UTC 
Why is this required for Cinder? Don't we do it for these VMs?
Comment 2 Daniel Erez 2016-06-14 13:42:19 UTC 
(In reply to Yaniv Dary from comment #1)
> Why is this required for Cinder? Don't we do it for these VMs?

We do it automatically, i.e. the special notes for the rest-api in the documentation are redundant.
Comment 3 Allon Mureinik 2016-06-15 07:47:05 UTC 
(In reply to Daniel Erez from comment #2)
> (In reply to Yaniv Dary from comment #1)
> > Why is this required for Cinder? Don't we do it for these VMs?
> 
> We do it automatically, i.e. the special notes for the rest-api in the
> documentation are redundant.

s/redundant/misleading/
Comment 5 Yaniv Lavi 2017-01-01 13:14:18 UTC 
I still see this issue in the 4.0 docs:
https://access.redhat.com/documentation/en/red-hat-virtualization/4.0/paged/virtual-machine-management-guide/69-snapshots

Can we please fix this?
Comment 6 Lucy Bopf 2017-01-11 04:12:56 UTC 
Assigning to Byron for review.
Comment 9 Byron Gravenorst 2017-05-03 01:55:30 UTC 
This content has been published to:

3.6: https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Virtualization/3.6/html/Virtual_Machine_Management_Guide/sect-Snapshots.html

4.0: https://access.redhat.com/documentation/en-us/red_hat_virtualization/4.0/html-single/virtual_machine_management_guide/#sect-Snapshots

4.1: https://access.redhat.com/documentation/en-us/red_hat_virtualization/4.1/html-single/virtual_machine_management_guide/#sect-Snapshots