[Engine-devel] failed to add host into cluster

[ouyang guohua]

On 11/22/2013 08:38 PM, ouyang guohua wrote:
> On 11/22/2013 07:52 PM, Dan Kenigsberg wrote:
>> On Thu, Nov 21, 2013 at 05:22:35PM +0100, Michal Skrivanek wrote:
>>>>> ok, that looks ok. Might be text parsing issue.
>>>>> And "virsh capabilities" output?
>>>>> what's the vdsm you're using?
>>>> virsh capabilities" output is a bit long, see attachment
>>>>
>>>> # rpm -q vdsm
>>>> vdsm-4.10.3-18.fc19.x86_64
>>> hmm, works well on my box
>>> try master, though this code didn't change since 2012
>>>
>>> the code in vdsm/caps.py is pretty straightforward:
>>>      caps = minidom.parseString(capabilities)
>>>      for archTag in caps.getElementsByTagName('arch'):
>>>          if archTag.getAttribute('name') == 'x86_64':
>>>              return [m.firstChild.data for m in archTag.childNodes
>>>                      if m.nodeName == 'machine']
>>>
>>>
>>> try to uninstall qemu Alpha support if it changes a thing or not...
>> As Michal noted, Vdsm parses the attached capabilities as it should.
>> Maybe it receives something else from libvirt. Could you add some
>> logging and report?
>>
>> diff --git a/vdsm/caps.py b/vdsm/caps.py
>> index 5599522..4daca46 100644
>> --- a/vdsm/caps.py
>> +++ b/vdsm/caps.py
>> @@ -136,6 +136,7 @@ def _getCpuTopology(capabilities):
>>   def _getEmulatedMachines(capabilities=None):
>>       if capabilities is None:
>>           capabilities = _getCapsXMLStr()
>> +    logging.debug('caps: %s', capabilities)
>>       caps = minidom.parseString(capabilities)
>>       for archTag in caps.getElementsByTagName('arch'):
>>           if archTag.getAttribute('name') == 'x86_64':
>>
>> Maybe you can add some logging
> ok, I will try it next Tuesday because I could not access the machine at
> this time.
my caps.py is different from above, so go to update the vdsm package, and the problem is gone
after updated vdsm-4.10.3-18.fc19.x86_64 to 4.13.0-180.git5daa7fa.fc19.x86_64

# vdsClient -s 0 getVdsCaps | grep -A 15 emulatedMachines

     emulatedMachines = ['pc',

                         'q35',

                         'isapc',

                         'pc-0.10',

                         'pc-0.11',

                         'pc-0.12',

                         'pc-0.13',

                         'pc-0.14',

                         'pc-0.15',

                         'pc-1.0',

                         'pc-1.1',

                         'pc-1.2',

                         'pc-1.3',

                         'none']

     guestOverhead = '65'

     hooks = {}


so, the problem maybe exists on the old vdsm package but fixed on the new package.


>
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel