no module named mock
Bug #1037422 reported by
Vincent Hou
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Cinder |
Fix Released
|
Medium
|
Vincent Hou | ||
Bug Description
I was running ./run_tests.sh under python-
Mock is listed in test-requires and install_venv.py doesn't define test_requires to be installed. Shall we add test_requires into install_venv.py?
Revision history for this message
| John Griffith (john-griffith) wrote | #1 |
| Changed in cinder: | |
| status: | New → Confirmed |
| importance: | Undecided → Medium |
| Changed in cinder: | |
| milestone: | none → folsom-rc1 |
| Changed in cinder: | |
| assignee: | nobody → Vincent Hou (houshengbo) |
Revision history for this message
| John Griffith (john-griffith) wrote | #2 |
| Changed in cinder: | |
| status: | Confirmed → Fix Committed |
| Changed in cinder: | |
| status: | Fix Committed → Fix Released |
| Changed in cinder: | |
| milestone: | folsom-rc1 → 2012.2 |
To post a comment you must log in.
Yes, adding test_requires to install_venv is probably a good idea. Keep consistency with novaclient and avoid confusion for folks that may be running venv with cinderclient for the first time.