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/207610554So ... 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