Bug 1315435
Summary: | vdsm-tool restore-nets fails due to invalid SMBIOS strings | ||||||
---|---|---|---|---|---|---|---|
Product: | [oVirt] vdsm | Reporter: | Jonathan Sherman <haviland> | ||||
Component: | Core | Assignee: | Martin Polednik <mpoledni> | ||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | sefi litmanovich <slitmano> | ||||
Severity: | high | Docs Contact: | |||||
Priority: | high | ||||||
Version: | 4.17.23 | CC: | bugs, danken, haviland, michal.skrivanek, slitmano, tjelinek | ||||
Target Milestone: | ovirt-3.6.5 | Flags: | michal.skrivanek:
ovirt-3.6.z?
rule-engine: planning_ack? danken: devel_ack+ rule-engine: testing_ack+ |
||||
Target Release: | 4.17.25 | ||||||
Hardware: | x86_64 | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | 3.6.5-1 | Doc Type: | Bug Fix | ||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2016-04-21 14:42:59 UTC | Type: | Bug | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | Virt | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
Jonathan Sherman
2016-03-07 17:49:20 UTC
Note: I've been working with the users mailing list and the following people were also engaged on this: Dan Kenigsberg <danken> Martin Polednik <mpolednik>, Yedidyah Bar David <didi>, mpoledni, edwardh Could you attach a binary form of the output of virsh -r nodedev-dumpxml computer > /tmp/computer.xml Tomas, we end up failing to restore networking, but the problem here is with the hostdev module, which does not behave when libvirt reports non-ascii data. Polednik is aware of this issue. Created attachment 1135298 [details]
Output of: virsh -r nodedev-dumpxml computer
(In reply to Dan Kenigsberg from comment #2) > Could you attach a binary form of the output of > > virsh -r nodedev-dumpxml computer > /tmp/computer.xml Attached! Comment on attachment 1135298 [details]
Output of: virsh -r nodedev-dumpxml computer
Note that this file was generated on a second device of the same hardware model which has the same issue.
To make sure that I understand the issue: this happens without any modification to the SMBIOS string? It seems more of a bug on the motherboard. If that is the case, we'll have to drop invalid fields for motherboards/devices with similar problems as we do not now the encoding of source string. Correct - this is an unmodified BIOS. I can't argue that Intel should have set those BIOS fields to valid strings, but they did not in the original or updated BIOS firmware. (In reply to Martin Polednik from comment #7) > To make sure that I understand the issue: this happens without any > modification to the SMBIOS string? It seems more of a bug on the motherboard. > > If that is the case, we'll have to drop invalid fields for > motherboards/devices with similar problems as we do not now the encoding of > source string. Let me know if there's any testing you'd like me to do on this. Verified with vdsm-4.17.25-0.el7ev.noarch. Forced an xml with non ascii chars as per the description on _parse_device_params method under /usr/share/vdsm/hostdev.py. Ran vdsClient -s 0 hostdevListByCaps to invoke call to the method. The non ascii chars are ignored and no exception is raised. if e.g. the xml is changes so product tag has: <product>���������������test������������������</product> then vdsClient -s 0 hostdevListByCap returns pair 'product': 'test' for that device. |