Change in vdsm[master]: hostdev: report additional information in 'scsi' device

danken at redhat.com danken at redhat.com
Sun May 8 13:50:34 UTC 2016


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 <mpolednik at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/56038
Reviewed-by: Milan Zamazal <mzamazal at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <fromani at 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 <mpolednik at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzamazal at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list