Allon Mureinik 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 <sgot...@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 Maor Lipchuk: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/25890 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife467feb8aad9711da5d6a5df305e50fe6bf51e9 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sergey Gotliv <sgot...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Daniel Erez <de...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgot...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
