From 143b3dc2864bec1f92eeed44a351e420ae68fd58 Mon Sep 17 00:00:00 2001 From: Ihar Hrachyshka Date: Tue, 24 Nov 2015 15:13:00 +0100 Subject: [PATCH] tox: pass TEMPEST_CONFIG_DIR envvar into api target environment Currently, API job skips all tests because TEMPEST_CONFIG_DIR is not set properly. The environment variable was hardcoded to /opt/stack/tempest/etc before, and since that's the location for tempest configuration files that was used in gate, it worked fine there. But after we were forced to remove the hardcoded value from the file due to regression in tox 2.2.0+ (If3742b6eabe295a5aaeb4901c55909a70c451698), it was neither hardcoded, nor passed from the external gate environment. Note that since tox 2.0.0, the tool does not pass all envvars into tox targets, but require explicit specification of the list of variables that should be allowed into the target environment, using passenv directive. So that's what the patch does: it allows TEMPEST_CONFIG_DIR to get into api tox target, which should be enough to make the API job avoid skipping all tests. Users are still required to set TEMPEST_CONFIG_DIR variable locally if they want to execute the suite locally. Closes-Bug: #1518632 Change-Id: Ia7dcc6785390a6012c7ba16a14573aa44f2b6fb5 --- tox.ini | 1 + 1 file changed, 1 insertion(+) diff --git a/tox.ini b/tox.ini index 23b70288a7b..acec80aaecf 100644 --- a/tox.ini +++ b/tox.ini @@ -21,6 +21,7 @@ commands = [testenv:api] basepython = python2.7 +passenv = {[testenv]passenv} TEMPEST_CONFIG_DIR setenv = {[testenv]setenv} OS_TEST_PATH=./neutron/tests/api OS_TEST_API_WITH_REST=1