Change in ovirt-engine[ovirt-engine-3.4]: engine: Send ifaceName and tpgt in connectStorageServer

iheim at redhat.com iheim at redhat.com
Sun Apr 13 14:10:48 UTC 2014


Itamar Heim has submitted this change and it was merged.

Change subject: engine: Send ifaceName and tpgt in connectStorageServer
......................................................................


engine: Send ifaceName and tpgt in connectStorageServer

Engine mistakenly sent initiatorName instead of ifaceName which overwrote
existing initiatorName configured in the host. Wrong initiatorName caused
host fail to perform iSCSI connection.

In case of tpgt the problem is terminology. Engine sent it as a "portal"
and its value is integer,  but from the iSCSI perspective portal is an
access point to one or more targets, while this integer actually
represents a tpgt - target portal group tag which is numeric representation
of which IP addresses a target is allowed to bind on.

Bug-Url: https://bugzilla.redhat.com/1078137
Change-Id: Ife467feb8aad9711da5d6a5df305e50fe6bf51e9
Signed-off-by: Sergey Gotliv <sgotliv at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/ConnectStorageServerVDSCommand.java
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Sergey Gotliv: Verified
  Allon Mureinik: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/26659
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ife467feb8aad9711da5d6a5df305e50fe6bf51e9
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
Gerrit-Owner: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list