Merge lp:~jshepher/nova/lp792156 into lp:~hudson-openstack/nova/trunk

[Justin Shepherd]

paramiko is not installed into the venv, but is required by smoketests/base.py. Added paramiko to tools/pip-requires.

[Vish Ishaya (vishvananda)]

good catch. LGTM. (admin smoketests also require nova_adminclient, perhaps it should go in the venv too?)

review: Approve

[Justin Shepherd (jshepher)]

I did not catch that one.. i guess i installed that package already on my laptop.

quick question.. this branch is at revno: 1121 (-22 behind trunk).. want me to delete branch and recreate?

Will also look at adding nova_adminclient.

[Devin Carlen (devcamcar)]

Vish is correct - we do require nova-adminclient for the admin tests currently. Can you please add it as well while we're at it?

[Justin Shepherd (jshepher)]

not a problem.. was in the middle of validating it as a missing dependency.

[lp:~jshepher/nova/lp792156]

1122. By Justin Shepherd on 2011-06-03

merged trunk

1123. By Justin Shepherd on 2011-06-03

added nova_adminclient to tools/pip-requires

[Justin Shepherd (jshepher)]

Added nova_adminclient to tools/pip-requires.. it looks like this gets installed as a dependency but is not explicitly defined.

[Josh Kearney (jk0)]

Looks good to me.

review: Approve

=== modified file 'tools/pip-requires'
--- tools/pip-requires	2011-05-27 20:31:18 +0000
+++ tools/pip-requires	2011-06-03 02:20:59 +0000
@@ -33,3 +33,5 @@
 coverage
 nosexcover
 GitPython
+paramiko
+nova_adminclient