Open Stack

On 10/27/20 02:08, Lee Yarwood wrote:
> On 26-10-20 13:32:20, Kashyap Chamarthy wrote:
>> On Mon, Oct 26, 2020 at 01:17:17PM +0100, Kashyap Chamarthy wrote:
>>
>> [...]
>>
>>> Great sleuthing so far.  (Not to mention the terribly cryptic orignal
>>> error message)
>>>
>>> As a small addendum, was talking to Dan Berrangé from libvirt and
>>> pointed to the openAuth() method ... and  he wrote a quick
>>> 'libvirt-python' unit test for the openAuth() method:
>>>
>>>      https://gitlab.com/libvirt/libvirt-python/-/merge_requests/28/diffs
>>>
>>> ... and it seems to be working as expected (based on the CI result):
>>>
>>>      https://gitlab.com/libvirt/libvirt-python/-/merge_requests/28/pipelines
>>>
>>> ... it seems to validate several operating system platforms (and Python
>>> versions), including Ubuntu 18.04:
>>>
>>>      https://gitlab.com/berrange/libvirt-python/-/pipelines/207610554
>>
>> So ... Dan was able to reproduce the problem; the culprit was the the
>> combination of thread pool, Python 3.6.x and type hinting; the above
>> unit test from Dan didn't make use of 'tpool'.  He says:
>>
>>      tpool + Python 3.6.x + type hinting is fubar
>>
>>      If I add 'tpool' into my new unit tests, then it crashes & burns
>>      with the error msg Mel got
>>
>>      
>> [...]
> 
> I've pushed https://review.opendev.org/#/c/759831/ based on the feedback
> from danpb, lets see if it resolves the issue.

Thanks to all for picking this up and running with it! I'm happy to see 
we've got a real fix for this and we don't have to revert the 
libvirt-python version bump.

Cheers,
-melanie