Tox fails when run in local environment

Bug #1828254 reported by Bart Wensley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Medium
Erich Cordoba

Bug Description

Brief Description
-----------------
Tox is no longer working in a local environment set up by following the instructions at https://docs.starlingx.io/contributor/build_guides/current/index.html

This has been broken since the switch to opendev.

Severity
--------
Minor: This only affects designers in their development environment - tox runs properly from zuul.

Steps to Reproduce
------------------
Set up your local build environment and then attempt to run tox for a component. For example:
cd stx-nfv/nfv
tox

Expected Behavior
------------------
Tox should run the unit tests

Actual Behavior
----------------
Tox fails because the repo names have changed. The error will be something like this:

ERROR: could not install deps [./nfv-client, ./nfv-common, ./nfv-plugins, ./nfv-vim, ../../fault/fm-api, iso8601, keyring, kombu, kubernetes, passlib, pecan, pyparsing, PyYAML, sqlalchemy, stevedore, wsme, ./nfv-tests/nfv_scenario_tests, fixtures, jinja2, mock, testtools, pylint]; v = InvocationError('/tmp/bwensley_nfvtox/pylint/bin/pip install -chttps://opendev.org/openstack/requirements/raw/upper-constraints.txt ./nfv-client ./nfv-common ./nfv-plugins ./nfv-vim ../../fault/fm-api iso8601 keyring kombu kubernetes passlib pecan pyparsing PyYAML sqlalchemy stevedore wsme ./nfv-tests/nfv_scenario_tests fixtures jinja2 mock testtools pylint (see /tmp/bwensley_nfvtox/pylint/log/pylint-1.log)', 1)

Reproducibility
---------------
Reproducible

System Configuration
--------------------
Development environment - does not affect the running system.

Branch/Pull Time/Commit
-----------------------
Fails since the cutover to opendev on about April 20.

Last Pass
---------
Prior to the cutover to opendev.

Timestamp/Logs
--------------
See above

Test Activity
-------------
Developer unit testing

Revision history for this message
Dean Troyer (dtroyer) wrote :

The long-term fix for this is to remove the need to have hard-coded paths in places like tox.ini to find dependencies.

Until then, updating the manifest to use the new repo names is the quickest approach however that will require finding the remaining locations where those repo/directory names are hard-coded in the build system and other places.

Bruce Jones (brucej)
Changed in starlingx:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Erich Cordoba (ericho)
Revision history for this message
Ghada Khalil (gkhalil) wrote :

Marking as release gating given this is a developer productivity issue

tags: added: stx.build
tags: added: stx.2.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to manifest (master)

Fix proposed to branch: master
Review: https://review.opendev.org/657903

Changed in starlingx:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to root (master)

Fix proposed to branch: master
Review: https://review.opendev.org/657910

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

Fix proposed to branch: master
Review: https://review.opendev.org/658203

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

Fix proposed to branch: master
Review: https://review.opendev.org/666412

Revision history for this message
Erich Cordoba (ericho) wrote :

The fix is ready but the build team thinks that it's better to integrate this change after release. Is this ok for the reporter fo this bug?

Revision history for this message
Bart Wensley (bartwensley) wrote :

If the build team things this is too risky to change in stx.2.0, then I will accept that. Note that I am not the one who decides which LPs gate release stx.2.0.

Revision history for this message
Ghada Khalil (gkhalil) wrote :

Changing the release tag to stx.3.0 as per above.

tags: added: stx.3.0
removed: stx.2.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nfv (master)

Change abandoned by Bart Wensley (<email address hidden>) on branch: master
Review: https://review.opendev.org/679997
Reason: The recent code restructuring activity fixed the issue so this change is no longer required.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on update (master)

Change abandoned by Bart Wensley (<email address hidden>) on branch: master
Review: https://review.opendev.org/658144
Reason: Abandoning this review since the change was already made under: https://review.opendev.org/#/c/681088/

Revision history for this message
Bart Wensley (bartwensley) wrote :

This has been fixed by the recent repo restructuring activity.

Changed in starlingx:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on manifest (master)

Change abandoned by Erich Cordoba (<email address hidden>) on branch: master
Review: https://review.opendev.org/657903

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on root (master)

Change abandoned by Erich Cordoba (<email address hidden>) on branch: master
Review: https://review.opendev.org/657910

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on integ (master)

Change abandoned by Erich Cordoba (<email address hidden>) on branch: master
Review: https://review.opendev.org/658203

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tools (master)

Change abandoned by Erich Cordoba (<email address hidden>) on branch: master
Review: https://review.opendev.org/666412

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.