Bug 1527047

Summary: Add portal to the popup dialog of iSCSI connections
Product: [oVirt] ovirt-engine Reporter: Maor <mlipchuk>
Component: Frontend.WebAdminAssignee: Maor <mlipchuk>
Status: CLOSED WONTFIX QA Contact: Pavel Stehlik <pstehlik>
Severity: low Docs Contact:
Priority: low    
Version: 4.2.0CC: amureini, bugs, ehildesh, gshereme, tnisan, ylavi
Target Milestone: ---Flags: sbonazzo: ovirt-4.3-
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: 2018-02-20 12:57:11 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:
Embargoed:
Attachments:
Description Flags
discover_iscsi
none
import_iscsi
none
connected_target_add_iscsi_storage none

Description Maor 2017-12-18 12:31:28 UTC
Created attachment 1369417 [details]
discover_iscsi

Description of problem:
The popup dialog of iSCSI connections should also reflect the portal of the iSCSI connection as part of the connection and the port

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


How reproducible:


Steps to Reproduce:
1. Try to add an iSCSI storage domain
2. Discover the targets
3.

Actual results:
Targets are reflected without portal

Expected results:
Targets should also be reflected with the portal

Additional info:
Should be fixed also for import iSCSI storage domain

Comment 1 Maor 2017-12-18 12:32:05 UTC
Created attachment 1369418 [details]
import_iscsi

Comment 2 Maor 2017-12-18 12:32:41 UTC
Created attachment 1369419 [details]
connected_target_add_iscsi_storage

Comment 3 Maor 2017-12-18 12:35:38 UTC
Greg, do you have any insight where will be the best place to add the portal to the pop up

Comment 4 Greg Sheremeta 2017-12-18 15:16:33 UTC
Totally a question for Eldan, but looks to me like it would just go under "port"

Comment 5 Eldan Hildesheim 2018-02-08 14:17:41 UTC
I Talked to Maor,
There is no use case for adding this field, so from my point of view, don't add it.
Do we need for login threw rest?

Comment 7 Eldan Hildesheim 2018-02-13 13:26:27 UTC
Same conclusion as before then, not necessary.

Comment 8 Maor 2018-02-20 12:57:11 UTC
Closed based on Comment 7