[ovirt-users] restore-nets failing (was: Fresh install failing (Hosted Engine))

Mon Mar 7 16:22:53 UTC 2016

On 07/03/16 11:09 -0500, Jonathan Sherman wrote:
>Thanks for your time on this Dan!
>
>The output from the hostdev looks like it may be unparseable, so I'm hoping
>this is the issue (and that it can be easily remedied).
>
>I've also create a log of the other items you asked for, available at:
>https://www.dropbox.com/s/qh0yw1ptpivpatm/typescript?dl=0
>
>[root at ovirt01 vdsm]# vdsm-tool restore-nets
><device>
>  <name>computer</name>
>  <capability type='system'>
>    <product>���������������������������������</product>
>    <hardware>
>      <vendor>���������������������������������</vendor>
>      <version>���������������������������������</version>
>      <serial>���������������������������������</serial>
>      <uuid>d6a3e3c1-c5cb-42e9-a54c-ff8d0df91722</uuid>
>    </hardware>
>    <firmware>
>      <vendor>Intel Corp.</vendor>
>      <version>FYBYT10H.86A.0052.2015.0923.1845</version>
>      <release_date>09/23/2015</release_date>
>    </firmware>
>  </capability>
></device>

That seems to be the issue. Even bigger issue is that we can
not skip this device easily, as it is the root of device tree and must
be present in database.

I can think of logging the exception but letting the call go through
and create a hook to fake a (minimal) device tree. Dan, what do you think?

mpolednik

>Traceback (most recent call last):
>  File "/usr/share/vdsm/vdsm-restore-net-config", line 429, in <module>
>    restore(args)
>  File "/usr/share/vdsm/vdsm-restore-net-config", line 387, in restore
>    _restore_sriov_numvfs()
>  File "/usr/share/vdsm/vdsm-restore-net-config", line 82, in
>_restore_sriov_numvfs
>    sriov_devices = _get_sriov_devices()
>  File "/usr/share/vdsm/vdsm-restore-net-config", line 54, in
>_get_sriov_devices
>    devices = hostdev.list_by_caps()
>  File "/usr/share/vdsm/hostdev.py", line 177, in list_by_caps
>    libvirt_devices = _get_devices_from_libvirt()
>  File "/usr/share/vdsm/hostdev.py", line 162, in _get_devices_from_libvirt
>    for device in libvirtconnection.get().listAllDevices(0))
>  File "/usr/share/vdsm/hostdev.py", line 162, in <genexpr>
>    for device in libvirtconnection.get().listAllDevices(0))
>  File "/usr/share/vdsm/hostdev.py", line 109, in _parse_device_params
>    devXML = etree.fromstring(device_xml)
>  File "/usr/lib64/python2.7/xml/etree/ElementTree.py", line 1300, in XML
>    parser.feed(text)
>  File "/usr/lib64/python2.7/xml/etree/ElementTree.py", line 1642, in feed
>    self._raiseerror(v)
>  File "/usr/lib64/python2.7/xml/etree/ElementTree.py", line 1506, in
>_raiseerror
>    raise err
>xml.etree.ElementTree.ParseError: not well-formed (invalid token): line 4,
>column 13
>Traceback (most recent call last):
>  File "/bin/vdsm-tool", line 219, in main
>    return tool_command[cmd]["command"](*args)
>  File "/usr/lib/python2.7/site-packages/vdsm/tool/restore_nets.py", line
>41, in restore_command
>    exec_restore(cmd)
>  File "/usr/lib/python2.7/site-packages/vdsm/tool/restore_nets.py", line
>54, in exec_restore
>    raise EnvironmentError('Failed to restore the persisted networks')
>EnvironmentError: Failed to restore the persisted networks
>
>On Mon, Mar 7, 2016 at 10:37 AM, Dan Kenigsberg <danken at redhat.com> wrote:
>
>> On Mon, Mar 07, 2016 at 03:16:06PM +0200, Yedidyah Bar David wrote:
>> > On Mon, Mar 7, 2016 at 3:01 PM, Jonathan Sherman <haviland at gmail.com>
>> wrote:
>> > > Didi -
>> > >
>> > > The vdsmd service error is what led me to the vdsm-tool restore-nets
>> > > concern.  See the chain of errors below (along with the other details
>> you
>> > > requested.)
>> >
>> > Thanks.
>> >
>> > Changing subject and adding Dan.
>>
>> > >
>> > > [root at ovirt01 log]# vdsm-tool restore-nets
>> > > Traceback (most recent call last):
>> > >   File "/usr/share/vdsm/vdsm-restore-net-config", line 429, in <module>
>> > >     restore(args)
>> > >   File "/usr/share/vdsm/vdsm-restore-net-config", line 387, in restore
>> > >     _restore_sriov_numvfs()
>> > >   File "/usr/share/vdsm/vdsm-restore-net-config", line 82, in
>> > > _restore_sriov_numvfs
>> > >     sriov_devices = _get_sriov_devices()
>> > >   File "/usr/share/vdsm/vdsm-restore-net-config", line 54, in
>> > > _get_sriov_devices
>> > >     devices = hostdev.list_by_caps()
>> > >   File "/usr/share/vdsm/hostdev.py", line 175, in list_by_caps
>> > >     libvirt_devices = _get_devices_from_libvirt()
>> > >   File "/usr/share/vdsm/hostdev.py", line 160, in
>> _get_devices_from_libvirt
>> > >     for device in libvirtconnection.get().listAllDevices(0))
>> > >   File "/usr/share/vdsm/hostdev.py", line 160, in <genexpr>
>> > >     for device in libvirtconnection.get().listAllDevices(0))
>> > >   File "/usr/share/vdsm/hostdev.py", line 107, in _parse_device_params
>> > >     devXML = etree.fromstring(device_xml)
>> > >   File "/usr/lib64/python2.7/xml/etree/ElementTree.py", line 1300, in
>> XML
>> > >     parser.feed(text)
>> > >   File "/usr/lib64/python2.7/xml/etree/ElementTree.py", line 1642, in
>> feed
>> > >     self._raiseerror(v)
>> > >   File "/usr/lib64/python2.7/xml/etree/ElementTree.py", line 1506, in
>> > > _raiseerror
>> > >     raise err
>> > > xml.etree.ElementTree.ParseError: not well-formed (invalid token):
>> line 4,
>> > > column 13
>> > > Traceback (most recent call last):
>> > >   File "/bin/vdsm-tool", line 219, in main
>> > >     return tool_command[cmd]["command"](*args)
>> > >   File "/usr/lib/python2.7/site-packages/vdsm/tool/restore_nets.py",
>> line
>> > > 41, in restore_command
>> > >     exec_restore(cmd)
>> > >   File "/usr/lib/python2.7/site-packages/vdsm/tool/restore_nets.py",
>> line
>> > > 54, in exec_restore
>> > >     raise EnvironmentError('Failed to restore the persisted networks')
>> > > EnvironmentError: Failed to restore the persisted networks
>>
>> Jonathan, thanks for this disturbing report. For some reason, Vdsm fails
>> to parse libvirt's output.
>>
>> Could you share your libvirt version as well the output of
>>     for d in `virsh -r nodedev-list`; do echo ====$d====; virsh -r
>> nodedev-dumpxml $d; done
>>
>> if all seems like valid xml, can you add the following print() and run
>> `restore-net` again?
>>
>> diff --git a/lib/vdsm/hostdev.py b/lib/vdsm/hostdev.py
>> index 6ba3189..9d22e9e 100644
>> --- a/lib/vdsm/hostdev.py
>> +++ b/lib/vdsm/hostdev.py
>> @@ -113,6 +113,7 @@ def _parse_device_params(device_xml):
>>      Process device_xml and return dict of found known parameters,
>>      also doing sysfs lookups for sr-iov related information
>>      """
>> +    print(device_xml)
>>      address_parser = {'pci': _parse_pci_address,
>>                        'scsi': _parse_scsi_address,
>>                        'usb_device': _parse_usb_address}
>>
>> Regards,
>> Dan.
>>