[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