run_tests without venv broken
Bug #1462922 reported by
Matthias Runge
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Dashboard (Horizon) |
Expired
|
Undecided
|
Unassigned |
Bug Description
as stated in review of https:/
this patch breaks tests without using venv. This is e.g. true, when packaging the whole thing, as builders don't have the connection to the internet
For me, this is a critical regression.
Changed in horizon: | |
status: | New → Confirmed |
Changed in horizon: | |
assignee: | nobody → Adriano (dritec) |
Changed in horizon: | |
assignee: | Adriano (dritec) → nobody |
To post a comment you must log in.
To continue the conversation, I repeat my comments here:
I was hoping you'd comment :). So, the goal here is to make a single call (`npm install`) bootstrap the entire development environment so that tests can be run, in a way that makes sense to both developers and packagers. That's why I have the if statement in the 'postinstall' hook, to detect the presence of two different venvs, and I would love any suggestions you have on how to detect whether to use .venv or the local environment.
Some constraints:
* Must work on infra, packager, and developer boxes (so must automagically detect where to get its dependencies from and/or whether to setup a venv).
* Must _only_ set up the environment. run_tests.sh -N actually runs tests, which we don't really want it to do in this case.
* Must not be using a deprecated way of building things: run_tests.sh is deprecated across openstack, Horizon's one of the few projects still using it. I would strongly prefer we not continue to overload something which should really go away.
In a perfect world, which I'm working towards, all the "Where do I get my dependencies from" should be resolved by npm and/or bower itself, automatically linking in appropriate sources if it detects them. So whatever steps we take, I'd like them to move us in that direction, though we'll have to put temporary shims in place as we get there.