Bug 878495 - [webadmin] Confusing text when importing a VM and same VM name already exists in environment - Clone
Summary: [webadmin] Confusing text when importing a VM and same VM name already exists...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine-webadmin-portal
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: 3.2.0
Assignee: Arik
QA Contact: Jiri Belka
URL:
Whiteboard: virt
Depends On:
Blocks: 915537
TreeView+ depends on / blocked
 
Reported: 2012-11-20 14:48 UTC by Jiri Belka
Modified: 2013-06-11 09:39 UTC (History)
7 users (show)

Fixed In Version: sf6
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-06-11 09:12:25 UTC
oVirt Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Current dialog (113.62 KB, image/png)
2013-01-14 09:03 UTC, Arik
no flags Details
Proposed dialog (124.17 KB, image/png)
2013-01-14 09:04 UTC, Arik
no flags Details
modified dialog for vm import conflict (138.69 KB, image/png)
2013-01-21 12:01 UTC, Arik
no flags Details
modified dialog for template import conflict (136.49 KB, image/png)
2013-01-21 12:02 UTC, Arik
no flags Details


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

Description Jiri Belka 2012-11-20 14:48:36 UTC
Description of problem:

When importing a VM and the VM name already exists in the environment (export-> then import it back), popup dialog says 'There is already a VM(s) with the same name (win2008_32bit) select: Don't Import/Clone'. The word 'Clone' is confusing, the content of the work 'Clone' is to make a copy of already existing thing for testing but in this context it is trying to tell us you can change name not to have duplicate VM names. Please invent other naming which would be more clear.

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

SI24.3

How reproducible:

Always.

Steps to Reproduce:
1. Make a VM
2. Export
3. Import while original VM still exists in the environment
  
Actual results:

Popup says 'Clone' in one of the possibilities.

Expected results:

Maybe better would be 'Rename imported' or something like that.
Additional info:

Comment 1 Arik 2013-01-14 09:03:42 UTC
Created attachment 678116 [details]
Current dialog

Comment 2 Arik 2013-01-14 09:04:52 UTC
Created attachment 678117 [details]
Proposed dialog

Comment 3 Arik 2013-01-14 09:16:06 UTC
Simon/Einav - please check the proposed dialog (attachment 678117 [details]), there are two changes comparing to the current dialog (attachment 6788116):

1. The first statement in the dialog - we open this kind of dialog for each VM with an already existing name, so I think "There already exists VM named: <vm_name>" is better than the current sentence: "Please select a name for the cloned VM(s) (<vm_name>)" + it doesn't contain the word "cloned"

2. "Cloned" is renamed to "Choose different name for the imported VM"

Is the proposed dialog OK ?

Comment 4 Simon Grinberg 2013-01-14 11:57:24 UTC
We are not just changing the name we also:
1. Change the UUID
2. Collapse snapshots into a new image. 

The above is a clone procedure same as you do when you clone from template. 
We need to make that clear, but I understand why just clone may be confusing.

So please add near the clone option saying "Import as a new cloned virtual machine (New name, IDs, and collapsed disk images)"

Please also explain that it's wither that or the suffix option - it's not clear

Comment 5 Jiri Belka 2013-01-14 12:24:28 UTC
The user trying to import his VM from export domain doesn't care about what is happening in background, that's why word 'clone' in this context is confusing. I'm OK with 'Proposed dialog' screenshot.

Comment 6 Simon Grinberg 2013-01-15 10:59:15 UTC
(In reply to comment #5)
> The user trying to import his VM from export domain doesn't care about what
> is happening in background, that's why word 'clone' in this context is
> confusing. I'm OK with 'Proposed dialog' screenshot.

Well product management is not OK with the change, the fact that it's satisfies your single use case does not mean it satisfies all.

Just to fix the sum up of changes, we are not just changing the name we also:
1. Change the UUID - <visible to advanced/api users 
2. Collapse snapshots into a new image - Immediately impacts storage if the VM is based on template and advance user api user will notice.
3. Changes the virtual NICs MAC addresses - Visible to the VM user and may impact ifcfg scripts. 

You raised a valid concern about clarity of the current dialogue however, it is a clone by definition and should be state as such, period. We can only do better by informing what exactly we are going to do - so change name is definitely a misleading term. 

The same user when he notices the changes will immediately call support with questions. Farther more the clarification is important since the user may not want the above to happen and may prefer not to import or to delete the existing VM before importing  

Let's say: 
"Import as cloned. Changing name, MAC addresses, and cloning all disks removing all snapshots."

UUID, is not interesting in this message, it's implied. 

Livnat, Ayal, is this all we do or did I miss something?

Comment 7 Arik 2013-01-21 12:01:51 UTC
Created attachment 684246 [details]
modified dialog for vm import conflict

Comment 8 Arik 2013-01-21 12:02:23 UTC
Created attachment 684247 [details]
modified dialog for template import conflict

Comment 9 Arik 2013-01-21 12:05:19 UTC
Simon, can you please check the two dialogs above (one for import VM conflict, and one for import Template conflict), if they're OK ?

Comment 10 Simon Grinberg 2013-01-21 17:20:36 UTC
(In reply to comment #9)
> Simon, can you please check the two dialogs above (one for import VM
> conflict, 

Import VM is OK.
I would drop the long description one line down to reduce the width of the dialogue. 

> and one for import Template conflict), if they're OK ?

We did not discuss import of existing templates in this thread.

This is a different discussion since there is an impact on VMs that the template is their parent image, please do not change anything yet and let's discuss over a new BZ - please describe current behavior for already existing:

1. Template with the same UUID 
2. Template with same name but different UUID  

We need to decide what to do with each - maybe we already have the right behavior.

Comment 12 Jiri Belka 2013-02-07 12:16:32 UTC
ok, sf6.

Comment 13 Itamar Heim 2013-06-11 09:12:25 UTC
3.2 has been released

Comment 14 Itamar Heim 2013-06-11 09:39:24 UTC
3.2 has been released


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