Bug 830541

Summary: webadmin [Import More than Once]: cannot uncheck the 'clone all vm's' checkbox in case the vm already exists in setup
Product: Red Hat Enterprise Virtualization Manager Reporter: Dafna Ron <dron>
Component: ovirt-engine-webadmin-portalAssignee: Gilad Chaplik <gchaplik>
Status: CLOSED CURRENTRELEASE QA Contact: Dafna Ron <dron>
Severity: high Docs Contact:
Priority: high    
Version: 3.1.0CC: acathrow, dfediuck, dyasny, ecohen, iheim, Rhev-m-bugs, ykaul
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard: virt
Fixed In Version: si10 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-04 20:07:48 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
screen shot none

Description Dafna Ron 2012-06-10 13:35:27 UTC
Created attachment 590747 [details]
screen shot

Description of problem:

if we try to import vm more then once we cannot uncheck the 'clone all vms' checkbox in the UI import dialoge. 

Version-Release number of selected component (if applicable):

si4

How reproducible:

100%

Steps to Reproduce:
1. try to import a vm more then once
2.
3.
  
Actual results:

when the vm already exists the 'clone all vms' checkbox is already check (it should be unchecked). and it can not be unchecked. 

Expected results:

1. checkbix should be unchecked as default 
2. we should be able to uncheck it

Additional info: screen shot

Comment 1 Gilad Chaplik 2012-06-10 13:52:08 UTC
why is it a bug? you can't import it unless you clone it (it's part of the design)

Comment 2 Dafna Ron 2012-06-10 14:00:03 UTC
1. from wiki: 

    Add 'clone all entities' check box to Import Entity Dialog 
        In case checked: 
            (in import VM) The 'collapse all snapshots' checkbox should be checked and disabled.
            A suffix textbox should be added (the suffix will be added to all VMs' names). 
    Add 'clone only existing entities' check box to Import Entity Dialog: 
        In case checked: 
            (in import VM) A label will be shown to user that indicates that the cloned VMs' snapshots will be collapsed.
            A suffix textbox should be added (the suffix will be added to all VMs names). 

2. working from test case:
https://tcms.engineering.redhat.com/case/146378/?from_plan=5756

case says that the checkbox should be unchecked by default. 

if only clone is allowed no one documented it.

Comment 3 Gilad Chaplik 2012-06-10 14:05:39 UTC
you're right, but this 'enhancement' is pretty straightforward (unless you object to it), I will change the docs accordingly, and we can close the bug.

Comment 7 Ayal Baron 2012-06-17 09:43:06 UTC
Looking at the screen shot, I have no idea what 'clone all vms' even means.
seems wrong (disks are automatically cloned due to the collapse all snapshots checkbox).

Comment 8 Gilad Chaplik 2012-06-19 13:00:10 UTC
we (Dafna, Dan & Simon) decided that the checkboxes will always we enabled.

Comment 10 Dafna Ron 2012-07-12 16:06:56 UTC
verified on si10. 
we are able to remove the tag and a vm already exists error is shown when removing the tag.