Change in vdsm[master]: Split getSessionInfo() out of getdeviScsiInfo()

ilvovsky at redhat.com ilvovsky at redhat.com
Sat Feb 4 16:13:49 UTC 2012


Igor Lvovsky has posted comments on this change.

Change subject: Split getSessionInfo() out of getdeviScsiInfo()
......................................................................


Patch Set 8: (2 inline comments)

....................................................
File vdsm/storage/iscsi.py
Line 342:         IscsiDevice
Maybe IscsiSession ?

....................................................
File vdsm/storage/multipath.py
Line 290:                     sessionInfo["portal"] = str(sessionInfo["portal"])
Sorry, but lost you here. Why all this?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6ba715a592f7d46e9d37c53d0abe1ea0c6fe4451
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list