Skip to content

Commit 9a1be77

Browse files
author
Ghanshyam Mann
committed
Pin tox<4.0.0 for <=stable/zed branch testing
Tox 4.0.0 has some incompatible changes, epecially more strict on allowlist_externals. Tempest recently changed allowlist_externals not to be *[1] causing the failure on jobs where lib/tempest failing to run the tempest as command in virtual env. ---------- venv: commands[0]> tempest verify-config -uro /tmp/tmp.qH5KgJHTF4 venv: failed with tempest is not allowed, use allowlist_externals to allow it ------ We do not need to test/fix the <=stable/zed branches with tox 4.0.0 and pinning them with the compatible tox version of the time stable brnaches were releaased is better way. This commit proposes: 1. Pinning the tox<4.0.0 for <=stable/ze branches testing 2. Workaround to unblock the master gate by pinning it <4.0.0 but we should make our testing compatible with tox 4.0.0 soon. Depends-On: https://review.opendev.org/c/openstack/devstack/+/867066 Related-Bug: #1999183 [1] https://review.opendev.org/c/openstack/tempest/+/865314 devstack based job started failing to run tempest command on venv. Change-Id: I9a138af94dedc0d8ce5a0d519d75779415d3c30b
1 parent fd502fe commit 9a1be77

2 files changed

Lines changed: 13 additions & 2 deletions

File tree

‎lib/tempest‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -779,7 +779,12 @@ function configure_tempest {
779779
# install_tempest() - Collect source and prepare
780780
function install_tempest {
781781
git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH
782-
pip_install 'tox!=2.8.0'
782+
# NOTE(gmann): Pinning tox<4.0.0 for stable/zed and lower. Tox 4.0.0
783+
# released after zed was released and has some incompatible changes
784+
# and it is ok not to fix the issues caused by tox 4.0.0 in stable
785+
# beanches jobs. We can continue testing the stable/zed and lower
786+
# branches with tox<4.0.0
787+
pip_install 'tox!=2.8.0,<4.0.0'
783788
pushd $TEMPEST_DIR
784789
# NOTE(gmann): checkout the TEMPEST_BRANCH in case TEMPEST_BRANCH
785790
# is tag name not master. git_clone would not checkout tag because

‎playbooks/tox/pre.yaml‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,10 @@
55
bindep_profile: test
66
bindep_dir: "{{ zuul_work_dir }}"
77
- test-setup
8-
- ensure-tox
8+
# NOTE(gmann): Pinning tox<4.0.0 for stable/zed and lower. Tox 4.0.0
9+
# released after zed was released and has some incompatible changes
10+
# and it is ok not to fix the issues caused by tox 4.0.0 in stable
11+
# beanches jobs. We can continue testing the stable/zed and lower
12+
# branches with tox<4.0.0
13+
- role: ensure-tox
14+
ensure_tox_version: "<4"

0 commit comments

Comments
 (0)