From 7ddbea25c4c443709809a8da92ecbb562cc3a62d Mon Sep 17 00:00:00 2001 From: Mark McLoughlin Date: Wed, 13 Mar 2013 06:57:19 +0000 Subject: [PATCH] Don't directly pip install tempest pip-requires This code: # Tempest doesn't satisfy its dependencies on its own, so # install them here instead. pip_install -r $TEMPEST_DIR/tools/pip-requires causes us to do: Found existing installation: oslo.config 1.1.0 Uninstalling oslo.config: so rather than 1.1.0 installed, we end up with 1.1.0b1 and everything goes kabloomy. It looks like this comment came from a time (commit 9a352daf) before tempest included proper install_requires (tempest commit 5abb253c3). It should be safe to delete this and allow setup_develop() do all the work just like the other projects. Change-Id: I869f727900bb08c72e5e8f4e84858a9c77049715 --- lib/tempest | 4 ---- 1 file changed, 4 deletions(-) diff --git a/lib/tempest b/lib/tempest index 9cc19ae0a8..85e643e474 100644 --- a/lib/tempest +++ b/lib/tempest @@ -266,10 +266,6 @@ function configure_tempest() { # install_tempest() - Collect source and prepare function install_tempest() { git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH - - # Tempest doesn't satisfy its dependencies on its own, so - # install them here instead. - pip_install -r $TEMPEST_DIR/tools/pip-requires } # init_tempest() - Initialize ec2 images