mos-infra-ci (gate-cinder-python27) doesn't use downstream dependencies

Bug #1655883 reported by Ivan Kolodyazhny
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Committed
High
Alexander Evseev

Bug Description

gate-cinder-python27 job for [1] uses os-brick from PyPI instead of downstream fork. It also doesn't honor Depends-On flag like it works in upstream [2].

[1] https://review.fuel-infra.org/#/c/29738/
[2] http://docs.openstack.org/infra/zuul/gating.html#cross-repository-dependencies

Ivan Kolodyazhny (e0ne)
Changed in mos:
importance: Undecided → High
Roman Vyalov (r0mikiam)
Changed in mos:
assignee: nobody → Alexander Evseev (aevseev-h)
milestone: none → 9.2
milestone: 9.2 → 9.3
Revision history for this message
Ivan Kolodyazhny (e0ne) wrote :

Roman, if we move this bug to 9.3, we have to move https://bugs.launchpad.net/mos/+bug/1653161 to 9.3 too

Revision history for this message
Alexander Evseev (aevseev) wrote :

Unit test jobs are using tox to run tests, tox uses pip to install requirements, pip installs requirements listed in requirements.txt and test-requirements.txt (if exists). If (test-)requirements.txt does not have additional info about module source (like git+https://…), then module is installed from PYPI.

Tag Depends-On intended ONLY to delay merge of change having dependency prior to required change, and it works only on gating stage when several changes are testing together.

Changed in mos:
status: New → Won't Fix
Revision history for this message
Vitaly Sedelnik (vsedelnik) wrote :

Reopened - we need to figure out how to run cinder unit test with os-brick from downstream repo or package.

Changed in mos:
status: Won't Fix → Confirmed
assignee: Alexander Evseev (aevseev-h) → Fuel CI (fuel-ci)
Anton Matveev (amatveev)
tags: added: customer-found sla1
Roman Vyalov (r0mikiam)
Changed in mos:
assignee: Fuel CI (fuel-ci) → Kirill Mashchenko (kmashchenko)
status: Confirmed → In Progress
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to openstack/requirements (stable/mitaka)

Related fix proposed to branch: stable/mitaka
Change author: Roman Vyalov <email address hidden>
Review: https://review.fuel-infra.org/30819

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to openstack/requirements (stable/mitaka)

Reviewed: https://review.fuel-infra.org/30819
Submitter: Alexander Evseev <email address hidden>
Branch: stable/mitaka

Commit: 55eb9f8f5b5009f9e53cb9c10c3c06dbca846a64
Author: Roman Vyalov <email address hidden>
Date: Tue Feb 14 11:16:12 2017

Remove os-brick requirements

Remove os-brick project from upper-constraints.txt

Change-Id: Id79650170d73e2250d61c223378f21352c701aba
Related-bug: #1655883

Changed in mos:
assignee: Kirill Mashchenko (kmashchenko) → Alexander Evseev (aevseev-h)
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to openstack/requirements (9.0/mitaka)

Related fix proposed to branch: 9.0/mitaka
Change author: Roman Vyalov <email address hidden>
Review: https://review.fuel-infra.org/30825

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to openstack/requirements (9.0/mitaka)

Reviewed: https://review.fuel-infra.org/30825
Submitter: Roman Vyalov <email address hidden>
Branch: 9.0/mitaka

Commit: f27bf9870a2ed2863d59a484683b48a3bb3be256
Author: Roman Vyalov <email address hidden>
Date: Tue Feb 14 12:20:51 2017

Remove os-brick requirements

Remove os-brick project from upper-constraints.txt

Change-Id: Id79650170d73e2250d61c223378f21352c701aba
Related-bug: #1655883
(cherry picked from commit 55eb9f8f5b5009f9e53cb9c10c3c06dbca846a64)

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to openstack/cinder (9.0/mitaka)

Related fix proposed to branch: 9.0/mitaka
Change author: Ivan Kolodyazhny <email address hidden>
Review: https://review.fuel-infra.org/30880

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to openstack/cinder (9.0/mitaka)

Reviewed: https://review.fuel-infra.org/30880
Submitter: Pkgs Jenkins <email address hidden>
Branch: 9.0/mitaka

Commit: 8aa990952b39f5462736462decb6c6fddb3365f2
Author: Ivan Kolodyazhny <email address hidden>
Date: Tue Feb 14 16:13:28 2017

Comment tests related to new os-brick library version

MOS Infra doesn't support run unit tests with downstream dependencies.
So this patch comments failed unit tests.

Change-Id: I9387a736a6ab38f6515f9480e7ec30687337415f
Related-Bug: #1655883

Revision history for this message
Roman Vyalov (r0mikiam) wrote :
Changed in mos:
status: In Progress → Fix Committed
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to openstack/cinder (9.0/mitaka)

Related fix proposed to branch: 9.0/mitaka
Change author: Ivan Kolodyazhny <email address hidden>
Review: https://review.fuel-infra.org/31191

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to openstack/cinder (9.0/mitaka)

Reviewed: https://review.fuel-infra.org/31191
Submitter: Pkgs Jenkins <email address hidden>
Branch: 9.0/mitaka

Commit: 0ccabab99071cbd7366bb1c162ffc15cb49b0441
Author: Ivan Kolodyazhny <email address hidden>
Date: Tue Feb 21 18:01:46 2017

Change skip decorators order to work with old os-brick

Change I9387a736a6ab38f6515f9480e7ec30687337415f added skip decorators
for unit-tests related to a new os-brick version. This change moves
these decorators on a top of the list to skip other decorators launch.

Change-Id: Ic4fc0b6051006d1a04ceabe02acbaa9bc5441103
Related-Bug: #1655883

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to openstack/cinder (mcp/newton)

Related fix proposed to branch: mcp/newton
Change author: Ivan Kolodyazhny <email address hidden>
Review: https://review.fuel-infra.org/33432

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to openstack/cinder (11.0/ocata)

Related fix proposed to branch: 11.0/ocata
Change author: Ivan Kolodyazhny <email address hidden>
Review: https://review.fuel-infra.org/34119

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to openstack/cinder (mcp/ocata)

Related fix proposed to branch: mcp/ocata
Change author: Ivan Kolodyazhny <email address hidden>
Review: https://review.fuel-infra.org/34759

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/cinder (mcp/ocata)

Change abandoned by Ivan Kolodyazhny <email address hidden> on branch: mcp/ocata
Review: https://review.fuel-infra.org/34759

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/cinder (11.0/ocata)

Change abandoned by Roman Podoliaka <email address hidden> on branch: 11.0/ocata
Review: https://review.fuel-infra.org/34119
Reason: we do not need 11.0/ocata anymore - use mcp/ocata instead

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/cinder (mcp/newton)

Change abandoned by Michael Dovgal <email address hidden> on branch: mcp/newton
Review: https://review.fuel-infra.org/33432

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to openstack/cinder (mcp/1.0/mitaka)

Related fix proposed to branch: mcp/1.0/mitaka
Change author: Ivan Kolodyazhny <email address hidden>
Review: https://review.fuel-infra.org/39534

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.