Dan Kenigsberg has submitted this change and it was merged.

Change subject: hostdev: report storage model as product
......................................................................


hostdev: report storage model as product

ovirt-engine frontend only displays values given by vendor and
product (because it is the most universal device description). Storage
is a special snowflake in this regard, as it has very nice identifier
hidden under 'model' key and no product identifier.

Reporting model instead of (empty) product leads to more understandable UI
data.

Change-Id: I8258988363faa00a9a32aa637dbed06ab82da55f
Bug-Url: https://bugzilla.redhat.com/1270581
Signed-off-by: Martin Polednik <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/56123
Reviewed-by: Francesco Romani <[email protected]>
Continuous-Integration: Jenkins CI
---
M lib/vdsm/hostdev.py
M tests/devices/data/Makefile.am
A tests/devices/data/block_sda_ssd.xml
M tests/hostdevTests.py
4 files changed, 35 insertions(+), 1 deletion(-)

Approvals:
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, approved
  Martin Polednik: Verified



-- 
To view, visit https://gerrit.ovirt.org/56123
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8258988363faa00a9a32aa637dbed06ab82da55f
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <[email protected]>
Gerrit-Reviewer: Milan Zamazal <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to