[openstack-dev] [stable][keystone] python3 goal progress and tox_install.sh removal

Ben Nemec openstack at nemebean.com
Thu Sep 6 20:28:38 UTC 2018



On 09/06/2018 03:01 PM, Lance Bragstad wrote:
> I'm noticing some odd cases with respect to the python 3 community goal 
> [0]. So far my findings are specific to keystone repositories, but I can 
> imagine this affecting other projects.
> 
> Doug generated the python 3 reviews for keystone repositories, including 
> the ones for stable branches. We noticed some issues with the ones 
> proposed to stable (keystoneauth, python-keystoneclient) and master 
> (keystonemiddleware). For example, python-keystoneclient's stable/pike 
> [1] and stable/ocata [2] branches are both failing with something like [3]:
> 
> ERROR: You must give at least one requirement to install (see "pip help 
> install")

We ran into this on the Oslo stable branches too. Instead of trying to 
migrate everything, we just tweaked tox_install.sh to avoid the problem: 
https://github.com/openstack/oslo.concurrency/commit/a009e7c86901473ac2273cb9ba604dc2a6b579d3

> 
> Both of those branches still use tox_install.sh [4][5]. Master, 
> stable/rocky, and stable/queens do not, which passed fine. It was 
> suggested that we backport patches to the failing branches that remove 
> tox_install.sh (similar to [6]). I've attempted to do this for 
> python-keystoneclient, keystonemiddleware, and keystoneauth.
> 
> The keystonemiddleware patches specifically are hitting a weird case, 
> where they either fail tests due to issues installing keystonemiddleware 
> itself, or pass tests and fail the requirements check. I'm guessing 
> (because I don't really fully understand the whole issue yet) this is 
> because keystonemiddleware has an optional dependency for tests and 
> somehow the installation process worked with tox_install.sh and doesn't 
> work with the new way we do things with pip and zuul.
> 
> I've attempted to remove tox_install.sh using several approaches with 
> keystonemiddleware master [7]. None of which passed both unit tests and 
> the requirements check.
> 
> I'm wondering if anyone has a definitive summary or context on 
> tox_install.sh and removing it cleanly for cases like 
> keystonemiddleware. Additionally, is anyone else noticing issues like 
> this with their stable branches?
> 
> [0] https://governance.openstack.org/tc/goals/stein/python3-first.html
> [1] https://review.openstack.org/#/c/597685/
> [2] https://review.openstack.org/#/c/597679/
> [3] 
> http://logs.openstack.org/85/597685/1/check/build-openstack-sphinx-docs/4f817dd/job-output.txt.gz#_2018-08-29_20_49_17_877448
> [4] 
> https://git.openstack.org/cgit/openstack/python-keystoneclient/tree/tools/tox_install.sh?h=stable/pike
> [5] 
> https://git.openstack.org/cgit/openstack/python-keystoneclient/tree/tools/tox_install.sh?h=stable/ocata
> [6] https://review.openstack.org/#/c/524828/3
> [7] https://review.openstack.org/#/c/599003/
> 
> 
> __________________________________________________________________________
> 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