Dan Kenigsberg has submitted this change and it was merged. Change subject: hostdev: report additional information in 'scsi' device ......................................................................
hostdev: report additional information in 'scsi' device SCSI capability in sysfs does not include enough information to identify the device in human readable way. We therefore fetch vendor and product from 'storage' capability, where these correspond to a physical device (if there is no physical device upstream, we ignore it). Another addition is udev path, that we require to fully construct the device in future. Change-Id: I789f49c9abca2dff1487acf3e8a04ae1407956f4 Bug-Url: https://bugzilla.redhat.com/1270581 Signed-off-by: Martin Polednik <mpoled...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/56038 Reviewed-by: Milan Zamazal <mzama...@redhat.com> Continuous-Integration: Jenkins CI Reviewed-by: Francesco Romani <from...@redhat.com> --- M lib/vdsm/hostdev.py M tests/devices/data/Makefile.am A tests/devices/data/scsi_1_0_0_0.xml A tests/devices/data/scsi_2_0_0_0.xml A tests/devices/data/scsi_generic_sg1.xml A tests/devices/data/scsi_host1.xml A tests/devices/data/scsi_host2.xml A tests/devices/data/scsi_target1_0_0.xml A tests/devices/data/scsi_target2_0_0.xml M tests/hostdevTests.py 10 files changed, 168 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Francesco Romani: Looks good to me, approved Martin Polednik: Verified Milan Zamazal: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/56038 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I789f49c9abca2dff1487acf3e8a04ae1407956f4 Gerrit-PatchSet: 9 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin Polednik <mpoled...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com> Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches