Bug 1393266

Summary:	Imported ISCSI storage domain name
Product:	[oVirt] ovirt-engine	Reporter:	Eyal Shenitzky <eshenitz>
Component:	BLL.Storage	Assignee:	Benny Zlotnik <bzlotnik>
Status:	CLOSED NOTABUG	QA Contact:	Raz Tamir <ratamir>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	4.0.5.5	CC:	amureini, bugs, eshenitz, mlipchuk, tnisan
Target Milestone:	ovirt-4.1.0-beta	Flags:	amureini: ovirt-4.1? eshenitz: planning_ack? eshenitz: devel_ack? eshenitz: testing_ack?
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:	2016-12-20 13:37:03 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Storage	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Eyal Shenitzky 2016-11-09 08:39:21 UTC

Description of problem:

When Importing ISCSI storage domain to the data center and set new name to 
the Imported storage domain, the name that eventually used is the original name of the imported storage domain and not the one that entered on the textbox.

Version-Release number of selected component (if applicable):
Engine - 4.0.5.5-0.1.el7ev
Vdsm - 4.18.15.2-1.el7ev.x86_64

How reproducible:
100%

Steps to Reproduce:
1.Create ISCSI storage domain
2.Detach the domain from 1
3.Remove the domain that detached in 2 from the data center without formatting   it.
4.Import the storage domain from 3 to the data center and enter new domain name.

Actual results:
Storage domain name is the original name

Expected results:
Storage domain name should be the one that set in the name textbox

Additional info:

Comment 1 Benny Zlotnik 2016-12-07 13:16:56 UTC

This is actually the desired behavior, it is possible to change the Imported Domain name using the editable column.

See comment here: https://bugzilla.redhat.com/show_bug.cgi?id=1195784#c5

Comment 2 Eyal Shenitzky 2016-12-11 09:45:11 UTC

In NFS we do have the option to change the import domain name using the name textBox.
If the textBox not in use why it isn't grayed out/presented to the user?
I think that the behavior should be consistent, set the name using the textBox.

Comment 3 Maor 2016-12-11 11:24:13 UTC

(In reply to Eyal Shenitzky from comment #2)
> In NFS we do have the option to change the import domain name using the name
> textBox.
> If the textBox not in use why it isn't grayed out/presented to the user?
> I think that the behavior should be consistent, set the name using the
> textBox.

NFS it is a different scenario since we can import only one NFS storage domain while in a block storage we can import multiple storage domains.
Also, when importing NFS we have no awareness of the storage domain name before the import while in block storage we do notice the storage domain name once we connect to the targets

Comment 4 Maor 2016-12-20 13:37:03 UTC

Based on https://bugzilla.redhat.com/1195784 and base on the reason we can not use the text box (See comment 3) closing this bug for now.
Please feel free to re-open if anyone disagree.