Bug 880742 - Import VM dialog ignores the Collapse All Snapshots checkbox when importing an existing VM
Summary: Import VM dialog ignores the Collapse All Snapshots checkbox when importing a...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine-webadmin-portal
Version: 3.1.0
Hardware: All
OS: All
unspecified
unspecified
Target Milestone: ---
: 3.3.0
Assignee: Tal Nisan
QA Contact: Gadi Ickowicz
URL:
Whiteboard: storage
Depends On:
Blocks: 872506 888309
TreeView+ depends on / blocked
 
Reported: 2012-11-27 17:29 UTC by Allon Mureinik
Modified: 2016-02-10 17:41 UTC (History)
11 users (show)

Fixed In Version: is1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-01-21 22:17:52 UTC
oVirt Team: Storage
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Engine log (7.02 KB, text/plain)
2012-11-27 17:29 UTC, Allon Mureinik
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 10828 0 None None None Never

Description Allon Mureinik 2012-11-27 17:29:46 UTC
Created attachment 652955 [details]
Engine log

Description of problem:
When importing a VM, the frontend forcibly sends collapseSnapshots=true if the imported VM already exists in the snapshot or Clone All is checked, regardless of whether the checkbox for Collapse All Snapshots is checked or not.

Version-Release number of selected component (if applicable):
Reproduced on upstream oVirt, change-id I7088b94cd716bb93b2a4813963a980f4eab9e597 (commit d099b397ef800917d06e173c93aba81cde3a5a9a).

How reproducible:
100%

Steps to Reproduce:
1. Create an NFS DC
2. Create a storage domain on the DC
3. Create a VM with a disk
4. Take a snapshot of the VM
5. Create an export domain
6. Export the VM
7. Import the same VM. Make sure that "Collapse All Snapshots" IS NOT CHECKED.
  
Actual results:
Although the checkbox was not checked, import is run with collapseAllSnapshots=ture, and the imported VM's snapshot is collapsed.

Expected results:
Need to understand if the described behaviour (i.e., overiding Collapse All Snapshots) is desired or not.
If it isn't - remove it.
If it is - mark the checkbox as checked, and grey it out, so the user will not have the misconception that he can control this behaviour.

Additional info:

Comment 1 Allon Mureinik 2012-11-28 17:00:44 UTC
Further analysis - this checkbox is a per VM property, not a global one, so the screen should probably be changed (a box per VM probably?).

Removing devel-ack and pm-ack so this bug can be re-eveluated and a direction decided.

Comment 6 Gadi Ickowicz 2013-08-12 10:20:12 UTC
Verified on is9.1.

Importing a VM without collapse snapshots imports the vm with the snapshot as expected. Attempting to import a VM more than once shows the checkboxes for Clone and Collapse for that vm as on and greyed out, and the import collapses the snapshots.

Comment 7 Itamar Heim 2014-01-21 22:17:52 UTC
Closing - RHEV 3.3 Released

Comment 8 Itamar Heim 2014-01-21 22:24:23 UTC
Closing - RHEV 3.3 Released


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