The cinder lower constraints CI doesn't work well

Bug #1801239 reported by Yikun Jiang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Yikun Jiang

Bug Description

The actual target of openstack-tox-lower-constraints is used the lower version package to run unit test.

But in Cinder CI,there was a hard code [1] to specify the upper-constraints.txt, that means we first install the package in upper-constraints.txt [2], the lower-constraints.txt doesn't work well.

The right thing is only install the lower in lower-constraints.txt to keep the package as lower version.

[1] https://github.com/openstack/cinder/blob/50a1ed7/tox.ini#L15
[2] http://logs.openstack.org/46/614446/1/check/openstack-tox-lower-constraints/d109789/job-output.txt.gz#_2018-10-31_09_38_56_802083
2018-10-31 09:38:56.802083 | ubuntu-xenial | /home/zuul/src/git.openstack.org/openstack/cinder$ /home/zuul/src/git.openstack.org/openstack/cinder/.tox/lower-constraints/bin/pip install -c/home/zuul/src/git.openstack.org/openstack/requirements/**upper-constraints.txt** --no-deps -e /home/zuul/src/git.openstack.org/openstack/cinder

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/614943

Changed in cinder:
assignee: nobody → Yikun Jiang (yikunkero)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/614943
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=3c2a22f75ca40c74e5b9edf4d49d66316d1c2504
Submitter: Zuul
Branch: master

commit 3c2a22f75ca40c74e5b9edf4d49d66316d1c2504
Author: Yikun Jiang <email address hidden>
Date: Fri Nov 2 11:05:32 2018 +0800

    Make lower constraints CI works well

    The actual target of openstack-tox-lower-constraints is using the lower
    version package to run unit test.

    But in current Cinder CI,there was a hard code to specify the
    upper-constraints.txt, that means we install the package in
    upper-constraints.txt, but the lower-constraints.txt doesn't work well.

    This patch try to fix this error, move the hard code to 'deps'.

    We also refresh the lower-constraints.txt and requirements.txt to make
    CI works well:
    1. Add libxml2-dev, libxslt-devel, libxslt1-dev to bindep.txt to make
    lxml can be built successfully, like what we do in other project. [0]
    2. Upgrade oslo.messaging to 6.4.0 to avoid "ACCESS_REFUSED
    - Login was refused using authentication mechanism AMQPLAIN." error.
    3. Upgrade oslo.utils to 3.34.0 to add a new method escape_ipv6 [1] to
    oslo_utils.netutils, which is needed by [2].
    4. Upgrade keystoneauth1 to 3.7.0 to fix "no such option split_loggers
    in group [service_user]" [3]

    [0] https://github.com/openstack/nova/blob/bb3ded3/bindep.txt#L17-L19
    [1] https://review.openstack.org/#/c/495166/
    [2] https://github.com/openstack/cinder/blob/fe421fe/cinder/volume/drivers/netapp/utils.py#L172
    [3] https://review.openstack.org/#/c/578008/

    Change-Id: Ide84d49c283c0ad179a5cfdb0392d34ee73d97ea
    Closes-bug: #1801239

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

This issue was fixed in the openstack/cinder 14.0.0.0rc1 release candidate.

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

Other bug subscribers

Remote bug watches

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