Sandro Bonazzola has posted comments on this change. Change subject: iscsi: add HE disk to the engine as direct lun ......................................................................
Patch Set 1: (2 comments) http://gerrit.ovirt.org/#/c/34783/1/src/plugins/ovirt-hosted-engine-setup/engine/add_disk.py File src/plugins/ovirt-hosted-engine-setup/engine/add_disk.py: Line 71: username=self.environment[ohostedcons.StorageEnv.ISCSI_USER], Line 72: password=self.environment[ohostedcons.StorageEnv.ISCSI_PASSWORD], Line 73: ) Line 74: disk = ovirtsdk.xml.params.Disk( Line 75: alias='direct_lun', has alias to be configurable? or fixed to something more specific to HE? Line 76: description=self.environment[ohostedcons.StorageEnv.IMAGE_DESC], Line 77: interface='virtio_scsi', Line 78: sgio='unfiltered', Line 79: format='raw', Line 74: disk = ovirtsdk.xml.params.Disk( Line 75: alias='direct_lun', Line 76: description=self.environment[ohostedcons.StorageEnv.IMAGE_DESC], Line 77: interface='virtio_scsi', Line 78: sgio='unfiltered', is sgio needed? Line 79: format='raw', Line 80: lun_storage=ovirtsdk.xml.params.Storage( Line 81: type_='iscsi', Line 82: logical_unit=[ -- To view, visit http://gerrit.ovirt.org/34783 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9c5640bde713fc4d09cf74a7037c62e833495934 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Doron Fediuck <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Lev Veyde <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Simone Tiraboschi <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Yedidyah Bar David <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
