Bug 830541 - webadmin [Import More than Once]: cannot uncheck the 'clone all vm's' checkbox in case the vm already exists in setup
Summary: webadmin [Import More than Once]: cannot uncheck the 'clone all vm's' checkbo...
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: x86_64
OS: Linux
high
high
Target Milestone: ---
: ---
Assignee: Gilad Chaplik
QA Contact: Dafna Ron
URL:
Whiteboard: virt
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-06-10 13:35 UTC by Dafna Ron
Modified: 2015-09-22 13:10 UTC (History)
7 users (show)

Fixed In Version: si10
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-12-04 20:07:48 UTC
oVirt Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
screen shot (46.97 KB, image/jpeg)
2012-06-10 13:35 UTC, Dafna Ron
no flags Details

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.


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