[openstack-dev] [nova] contextlib.nested and Python3 failing

Sylvain Bauza sbauza at redhat.com
Wed Aug 19 23:59:26 UTC 2015



Le 19/08/2015 16:51, Sylvain Bauza a écrit :
> Hi,
>
> I was writing some tests so I added a contextlib.nested to a checked 
> TestCase [1]. Unfortunately, contextlib.nested is no longer available 
> in Python3 and there is no clear solution on how to provide a 
> compatible import for both python2 and python3:
>  - either providing a python3 compatible behaviour by using 
> contextlib.ExitStack but that class is not available in Python 2
>  - or provide contextlib2 for python2 (and thus adding it to the 
> requirements)
>
> That sounds really disruptive and blocking as we are close to the 
> FeatureFreeze. Many other users of contextlib.nested are not impacted 
> by the job because it excludes all of them but since the test I'm 
> changing is part of the existing validated tests, that leaves Jenkins 
> -1'ing my change.
>
> Of course, a 3rd solution would consist of excluding my updated test 
> from the python3 check but I can hear others yelling at that :-)
>
> Ideas appreciated.
>

So, I just saw there is actually already a solution for that here:
https://github.com/openstack/nova/blob/master/nova/test.py#L72-L78

beer_count['dims']++

Thanks,

> -Sylvain
>
> [1] 
> https://review.openstack.org/#/c/199205/18/nova/tests/unit/scheduler/test_rpcapi.py,cm
>
>
> __________________________________________________________________________ 
>
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: 
> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev




More information about the OpenStack-dev mailing list