[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[nova][keystone][neutron][kuryr][requirements] breaking tests with new library versions

On 8/21/19 3:44 PM, Eric Fried wrote:
>>>> NOVA:
>>>> lxml===4.4.1        nova tests fail https://bugs.launchpad.net/nova/+bug/1838666
> Sean Mooney agreed to take a look at this one.
>>>> websockify===0.9.0  tempest test failing
> This is now known as bug 1840788 [1].
> I did some initial investigation and tried to fix it. Turned out my fix
> was already part of another change. But it also turns out that that fix
> is only part of the solution. See the bug report for (links to) details.
> At this point, it needs someone who understands what test_novnc [2] is
> actually trying to do. Because I don't.
> Anyone?

 From the test run on your patch [3], it looks like we're going to need 
a change in nova as well in nova/console/websocketproxy.py [4]:

   AttributeError: module 'websockify' has no attribute 'WebSocketServer'

It looks like the WebSocketServer class got moved from 
websockify.websocket to websockify.websocketserver in v0.9.0:


and thus is no longer accessible via the top level 'websockify' module 
as a result:


Since this is a change to upper-constraints to allow v0.9.0, we will 
need a way for it to work with both module layouts, yeah?


[3] https://review.opendev.org/677798

> Thanks,
> efried
> [1] https://bugs.launchpad.net/nova/+bug/1840788
> [2]
> https://opendev.org/openstack/tempest/src/branch/master/tempest/api/compute/servers/test_novnc.py#L180