Yoav Kleinberger has posted comments on this change. Change subject: clarify why scsiKey parameter is not passed on ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/28560/1//COMMIT_MSG Commit Message: Line 6: Line 7: clarify why scsiKey parameter is not passed on Line 8: Line 9: In this patch I add comments to make plain that the scsiKey parameter is Line 10: a legacy from older code. This will hopefully save time for the next guy > next person? the next person to wander why this parameter exists in the functional tests for example. That's what happened to me. Line 11: who goes on this wild goose chase through the code. Line 12: Line 13: Change-Id: I5edc98058f2332deb6ee040879b4f7c635aefbc2 -- To view, visit http://gerrit.ovirt.org/28560 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5edc98058f2332deb6ee040879b4f7c635aefbc2 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yoav Kleinberger <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> Gerrit-Reviewer: Yoav Kleinberger <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
