Change in vdsm[ovirt-3.4]: hsm: Distinguish between iface and initiator when creating i...

ybronhei at redhat.com ybronhei at redhat.com
Thu Apr 10 12:42:03 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: hsm: Distinguish between iface and initiator when creating iSCSI iface
......................................................................


hsm: Distinguish between iface and initiator when creating iSCSI iface

VDSM mistakely assumed that iface name and iface initiator name are the
same, so if admin configured specific initiator name in the host its
value is mistakenly overwritten by ifaceName in the iface file. In such
a case the host fails to establish the iSCSI connection with the target.

Change-Id: I2ea42e63e3e09babaa283c4e97e341ff259a6945
Bug-Url: https://bugzilla.redhat.com/1078137
Signed-off-by: Sergey Gotliv <sgotliv at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/25628
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/26656
Reviewed-by: Allon Mureinik <amureini at redhat.com>
---
M vdsm/storage/hsm.py
1 file changed, 4 insertions(+), 3 deletions(-)

Approvals:
  Federico Simoncelli: Looks good to me, approved
  Sergey Gotliv: Verified
  Allon Mureinik: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2ea42e63e3e09babaa283c4e97e341ff259a6945
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.4
Gerrit-Owner: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list