Dan Kenigsberg has submitted this change and it was merged. Change subject: dmidecode: Handle missing values ......................................................................
dmidecode: Handle missing values Make all hwInfo fields in API optional, and change property retrieval so it cannot fail, also, get rid of all keys that contain no actual information. The engine will assume that all the missing keys are not available and report/display an according message (rather than sending 'Not Specified') Change-Id: I979518e1d0c0c882fe98fd5aee43c0d50ab17e14 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1089393 Signed-off-by: Yaniv Bronhaim <ybron...@redhat.com> Signed-off-by: Dima Kuznetsov <dkuzn...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/26942 Reviewed-by: Dan Kenigsberg <dan...@redhat.com> --- M vdsm/dmidecodeUtil.py M vdsm_api/vdsmapi-schema.json 2 files changed, 13 insertions(+), 11 deletions(-) Approvals: Yaniv Bronhaim: Looks good to me, but someone else must approve Dima Kuznetsov: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26942 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I979518e1d0c0c882fe98fd5aee43c0d50ab17e14 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Dima Kuznetsov <dkuzn...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches