Merge lp:~jshepher/nova/lp780276 into lp:~hudson-openstack/nova/trunk
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Paul Voccio | ||||
Approved revision: | 1062 | ||||
Merged at revision: | 1063 | ||||
Proposed branch: | lp:~jshepher/nova/lp780276 | ||||
Merge into: | lp:~hudson-openstack/nova/trunk | ||||
Diff against target: |
20 lines (+2/-0) 2 files modified
Authors (+1/-0) tools/pip-requires (+1/-0) |
||||
To merge this branch: | bzr merge lp:~jshepher/nova/lp780276 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paul Voccio (community) | Approve | ||
Ed Leafe (community) | Approve | ||
Jay Pipes (community) | Approve | ||
Review via email: mp+60448@code.launchpad.net |
Commit message
Fix for #780276 (run_tests.sh fails test_authors_
Description of the change
Bug Message:
run_tests.sh fails test_authors_
Error Message:
=======
ERROR: test_authors_
-------
Traceback (most recent call last):
File "/opt/nova-
import git
ImportError: No module named git
-------
Steps to reproduce error:
# git clone https:/
# cd /opt/nova
# python tools/install_
# source .nova-venv/
# ./run_tests.sh
Proposed Solution:
Added GitPython to [install_
lgtm.
btw, Justin, this:
./run_tests.sh -V
is a shortcut for:
# python tools/install_ venv.py bin/activate
# source .nova-venv/
# ./run_tests.sh
Cheers!
jay