tox bindep environment looks for 'test' profile only
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Fix Released
|
Low
|
Brian Rosmaita | ||
os-brick |
Fix Released
|
Undecided
|
Brian Rosmaita |
Bug Description
Observed this while testing out the pdf-docs generation patch.
Suppose I ask tox to check my bindeps for the 'doc' profile:
$ tox -e bindep -- doc
bindep installed: bindep=
bindep run-test-pre: PYTHONHASHSEED=
bindep runtests: commands[0] | bindep test
______________ summary _______
bindep: commands succeeded
congratulations :)
It says everything is fine. But everything is not fine, I am missing some bindeps. You can tell if you run bindep directly:
$ .tox/bindep/
Missing packages:
python3-
The problem is that the bindep tox env is hard-coded to always select the 'test' profile. This makes it useless for testing any other profile.
Changed in python-cinderclient: | |
status: | New → Triaged |
assignee: | nobody → Brian Rosmaita (brian-rosmaita) |
Changed in cinder: | |
importance: | Undecided → Low |
Changed in python-cinderclient: | |
importance: | Undecided → Low |
no longer affects: | python-cinderclient |
Changed in cinder: | |
status: | Triaged → In Progress |
Changed in os-brick: | |
assignee: | nobody → Brian Rosmaita (brian-rosmaita) |
status: | New → In Progress |
Reviewed: https:/ /review. opendev. org/679124 /git.openstack. org/cgit/ openstack/ os-brick/ commit/ ?id=a8ee3a1b005 64cd5585f7b532a ea2aabd1f1bd5a
Committed: https:/
Submitter: Zuul
Branch: master
commit a8ee3a1b00564cd 5585f7b532aea2a abd1f1bd5a
Author: Brian Rosmaita <email address hidden>
Date: Wed Aug 28 15:13:12 2019 -0400
Fix tox 'bindep' environment
tox.ini is hard-coded to always use the 'test' profile for bindep;
instead, let tox get the profile from the posargs
Closes-bug: #1841836 0037f5f9aea9417 bc8371bfd91
Change-Id: I24bfb313cf28b2