From 3c1e4ba2e38f21c4686abe2e078301855e1d4216 Mon Sep 17 00:00:00 2001 From: Michael Johnson Date: Wed, 14 Mar 2018 09:49:29 -0700 Subject: [PATCH] Remove tox_install.sh Fully removing tox_install.sh as it is no longer needed with zuulv3. Change-Id: I72745f088a92010e43d7c078f8de05d529d20930 --- requirements.txt | 1 + tools/tox_install.sh | 72 -------------------------------------------- tox.ini | 7 +++-- 3 files changed, 5 insertions(+), 75 deletions(-) delete mode 100755 tools/tox_install.sh diff --git a/requirements.txt b/requirements.txt index 6fc9b9dda..09cd94dc4 100644 --- a/requirements.txt +++ b/requirements.txt @@ -7,6 +7,7 @@ eventlet!=0.18.3,!=0.20.1,<0.21.0,>=0.18.2 # MIT requests>=2.14.2 # Apache-2.0 netaddr>=0.7.18 # BSD neutron-lib>=1.13.0 # Apache-2.0 +neutron>=12.0.0 # Apache-2.0 SQLAlchemy!=1.1.5,!=1.1.6,!=1.1.7,!=1.1.8,>=1.0.10 # MIT alembic>=0.8.10 # MIT six>=1.10.0 # MIT diff --git a/tools/tox_install.sh b/tools/tox_install.sh deleted file mode 100755 index 946230554..000000000 --- a/tools/tox_install.sh +++ /dev/null @@ -1,72 +0,0 @@ -#!/usr/bin/env bash - -# Many of neutron's repos suffer from the problem of depending on neutron, -# but it not existing on pypi. - -# This wrapper for tox's package installer will use the existing package -# if it exists, else use zuul-cloner if that program exists, else grab it -# from neutron master via a hard-coded URL. That last case should only -# happen with devs running unit tests locally. - -# From the tox.ini config page: -# install_command=ARGV -# default: -# pip install {opts} {packages} - -ZUUL_CLONER=/usr/zuul-env/bin/zuul-cloner -BRANCH_NAME=master -neutron_installed=$(echo "import neutron" | python 2>/dev/null ; echo $?) -NEUTRON_DIR=$HOME/neutron -local ZUULV3_PROJECT_DIR=$HOME/src/git.openstack.org/openstack/$project - -set -e -set -x - -install_cmd="pip install -c$1" -shift - -# The devstack based functional tests have neutron checked out in -# $NEUTRON_DIR on the test systems - with the change to test in it. -# Use this directory if it exists, so that this script installs the -# neutron version to test here. -# Note that the functional tests use sudo to run tox and thus -# variables used for zuul-cloner to check out the correct version are -# lost. -if [ -d "$ZUULV3_PROJECT_DIR" ]; then - echo "FOUND Neutron code at $ZUULV3_PROJECT_DIR - using" - $install_cmd -U -e $ZUULV3_PROJECT_DIR -elif [ -d "$NEUTRON_DIR" ]; then - echo "FOUND Neutron code at $NEUTRON_DIR - using" - $install_cmd -U -e $NEUTRON_DIR -elif [ $neutron_installed -eq 0 ]; then - echo "ALREADY INSTALLED" > /tmp/tox_install.txt - location=$(python -c "import neutron; print(neutron.__file__)") - echo "ALREADY INSTALLED at $location" - - echo "Neutron already installed; using existing package" -elif [ -x "$ZUUL_CLONER" ]; then - echo "ZUUL CLONER" > /tmp/tox_install.txt - # Make this relative to current working directory so that - # git clean can remove it. We cannot remove the directory directly - # since it is referenced after $install_cmd -e. - mkdir -p .tmp - NEUTRON_DIR=$(/bin/mktemp -d -p $(pwd)/.tmp) - pushd $NEUTRON_DIR - $ZUUL_CLONER --cache-dir \ - /opt/git \ - --branch $BRANCH_NAME \ - git://git.openstack.org \ - openstack/neutron - cd openstack/neutron - $install_cmd -e . - popd -else - echo "PIP HARDCODE" > /tmp/tox_install.txt - if [ -z "$NEUTRON_PIP_LOCATION" ]; then - NEUTRON_PIP_LOCATION="git+https://git.openstack.org/openstack/neutron@$BRANCH_NAME#egg=neutron" - fi - $install_cmd -U -e ${NEUTRON_PIP_LOCATION} -fi - -$install_cmd -U $* -exit $? diff --git a/tox.ini b/tox.ini index 298ece4b8..1531c50d0 100644 --- a/tox.ini +++ b/tox.ini @@ -11,9 +11,10 @@ setenv = VIRTUAL_ENV={envdir} OS_LOG_CAPTURE={env:OS_LOG_CAPTURE:1} OS_LOG_DEFAULTS={env:OS_LOG_DEFAULTS:""} usedevelop = True -install_command = - {toxinidir}/tools/tox_install.sh {env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages} -deps = -r{toxinidir}/test-requirements.txt +install_command = pip install {opts} {packages} +deps = -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} + -r{toxinidir}/requirements.txt + -r{toxinidir}/test-requirements.txt commands = stestr run {posargs} stestr slowest