fuel-web wrong constraints in nailgun/requirements.txt

Bug #1685783 reported by Ivan Suzdal
28
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Denis Meltsaykin

Bug Description

Detailed bug description:
It seems like CR [0] brought wrong constraints against of mitaka global requirements. In the upstream [1] python-requests is constrained with '>=2.8.1,!=2.9.0' condition, in the fuel-web it's '!=2.12.2,!=2.13.0,>=2.10.0'. Besides this change broke nailgun_syncdb [2] (mirantis repo has python-requests version 2.8.1).

[0] https://review.openstack.org/#/c/456832/
[1] https://github.com/openstack/requirements/blob/stable/mitaka/global-requirements.txt#L221
[2] http://paste.openstack.org/show/607660/

Changed in fuel:
milestone: 9.x-updates → 9.2-mu-2
status: New → Confirmed
importance: Undecided → Critical
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/459284

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (stable/mitaka)

Change abandoned by Denis V. Meltsaykin (<email address hidden>) on branch: stable/mitaka
Review: https://review.openstack.org/459284
Reason: Abandoned in favor of: https://review.openstack.org/#/c/455731

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/mitaka)

Reviewed: https://review.openstack.org/455731
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=11430473db01af7647d8a2e71183c4d76590d475
Submitter: Jenkins
Branch: stable/mitaka

commit 11430473db01af7647d8a2e71183c4d76590d475
Author: Denis V. Meltsaykin <email address hidden>
Date: Tue Apr 11 17:27:23 2017 +0300

    Use upper constraints for mitaka

    This commit makes two changes:
            * make use of upper constraints for mitaka in tox to get rid
              of manully pinned requirements
            * get rid of the assert_called_once() call, which was never
              present in mock
            * get rid of the assert_called() call, which is not present
              in mock 1.3.0

    Closes-Bug: #1685783
    Change-Id: I9bb9c7c90ad8f3ddc14f1c6709376acc3b5e3395

tags: added: in-stable-mitaka
Changed in fuel:
status: Confirmed → Fix Committed
Revision history for this message
TatyanaGladysheva (tgladysheva) wrote :

Verified on 9.2 + mu2 updates.

Latest BVT runs with fix are passed (since #1256 snapshot):
https://product-ci.infra.mirantis.net/job/9.x.main.ubuntu.bvt_2/1259/

Changed in fuel:
status: Fix Committed → Fix Released
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.