From f97c2da08834c23b4f923e685dfeeddbe5b13f47 Mon Sep 17 00:00:00 2001 From: Dirk Mueller Date: Mon, 5 Dec 2016 09:28:16 +0100 Subject: [PATCH] Add Constraints support Adding constraints support to clients is slightly more complex than services as the clients themselves are listed in upper-constraints.txt which leads to errors that you can't install a specific version and a constrained version. This change adds constraints support by also adding a helper script to edit the constraints to remove monasca-ui. Change-Id: Ie2f4e3ed3e309b1176ec8bffa5466da25345fd64 --- tools/tox_install.sh | 30 ++++++++++++++++++++++++++++++ tox.ini | 13 ++++++++----- 2 files changed, 38 insertions(+), 5 deletions(-) create mode 100755 tools/tox_install.sh diff --git a/tools/tox_install.sh b/tools/tox_install.sh new file mode 100755 index 00000000..e61b63a8 --- /dev/null +++ b/tools/tox_install.sh @@ -0,0 +1,30 @@ +#!/usr/bin/env bash + +# Client constraint file contains this client version pin that is in conflict +# with installing the client from source. We should remove the version pin in +# the constraints file before applying it for from-source installation. + +CONSTRAINTS_FILE="$1" +shift 1 + +set -e + +# NOTE(tonyb): Place this in the tox enviroment's log dir so it will get +# published to logs.openstack.org for easy debugging. +localfile="$VIRTUAL_ENV/log/upper-constraints.txt" + +if [[ "$CONSTRAINTS_FILE" != http* ]]; then + CONSTRAINTS_FILE="file://$CONSTRAINTS_FILE" +fi +# NOTE(tonyb): need to add curl to bindep.txt if the project supports bindep +curl "$CONSTRAINTS_FILE" --insecure --progress-bar --output "$localfile" + +pip install -c"$localfile" openstack-requirements + +# This is the main purpose of the script: Allow local installation of +# the current repo. It is listed in constraints file and thus any +# install will be constrained and we need to unconstrain it. +edit-constraints "$localfile" -- "$CLIENT_NAME" + +pip install -c"$localfile" -U "$@" +exit $? diff --git a/tox.ini b/tox.ini index f63bb113..40eca371 100644 --- a/tox.ini +++ b/tox.ini @@ -1,18 +1,21 @@ [tox] envlist = py27,pep8 -minversion = 1.6 +minversion = 2.0 skipsdist = True [testenv] usedevelop = True -install_command = pip install -U {opts} {packages} -setenv = VIRTUAL_ENV={envdir} +setenv = + VIRTUAL_ENV={envdir} + BRANCH_NAME=master + CLIENT_NAME=monasca-ui deps = -r{toxinidir}/test-requirements.txt -r{toxinidir}/requirements.txt +install_command = {toxinidir}/tools/tox_install.sh {env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages} whitelist_externals = find commands = - find . -type f -name "*.pyc" -delete - /bin/bash run_tests.sh -N {posargs} + find . -type f -name "*.pyc" -delete + /bin/bash run_tests.sh -N {posargs} [testenv:pep8] commands = /bin/bash run_tests.sh -N --pep8