python-watcherclient

tox runs are not using upper-constraints

Bug #1828698 reported by Matt Riedemann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-watcherclient
Fix Released
Undecided
Matt Riedemann

Bug Description

As seen here: https://review.opendev.org/#/c/657375/

the py27 job fails because it's installing a too-high version of sphinx which doesn't support py27:

http://logs.openstack.org/75/657375/4/check/openstack-tox-py27/bda1455/ara-report/result/3fce905e-3aa1-413b-8f36-9c478a0d8d12/

Collecting sphinx!=1.6.6,!=1.6.7,>=1.6.5 (from -r /home/zuul/src/opendev.org/openstack/python-watcherclient/test-requirements.txt (line 12))
  Downloading http://mirror.sjc1.vexxhost.openstack.org/wheel/ubuntu-18.04-x86_64/sphinx/Sphinx-2.0.1-py2.py3-none-any.whl (3.2MB)
ERROR: Sphinx requires Python '>=3.5' but the running Python is 2.7.15

Matt Riedemann (mriedem)
Changed in python-watcherclient:
status: New → Confirmed
Revision history for this message
Matt Riedemann (mriedem) wrote :

Fix: https://review.opendev.org/658567

Changed in python-watcherclient:
status: Confirmed → In Progress
assignee: nobody → Matt Riedemann (mriedem)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-watcherclient (master)

Reviewed: https://review.opendev.org/658567
Committed: https://git.openstack.org/cgit/openstack/python-watcherclient/commit/?id=2cab116e305b798dc5c5e3b97528c78e9cfb77f4
Submitter: Zuul
Branch: master

commit 2cab116e305b798dc5c5e3b97528c78e9cfb77f4
Author: Matt Riedemann <email address hidden>
Date: Sat May 11 18:45:44 2019 -0400

    Use upper-constraints in tox runs

    The upper-constraints file wasn't being enforced
    when installing dependencies in tox runs which can
    lead to failures like installing Sphinx>2.0 in py27
    where it's not supported.

    This is loosely based on novaclient change
    I8be883215f27abb58d15b85e8542cbdf32000bac for the
    same kind of issue.

    Note there are other things that should be done
    in here regarding requirements cleanup, like the
    sphinx dependnecies should move to a
    doc/requirements.txt file to separate them from
    unit test runs, but that can be dealt with in a
    follow up change.

    Change-Id: I359a35af5b653299110a8d0faddb255da0b10118
    Closes-Bug: #1828698

Changed in python-watcherclient:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/python-watcherclient 2.3.0

This issue was fixed in the openstack/python-watcherclient 2.3.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.