From 15f9f1777e2205e8fe89c6eb35915c5e14ee6af7 Mon Sep 17 00:00:00 2001 From: manchandavishal Date: Fri, 6 Jan 2023 11:54:54 +0530 Subject: [PATCH] Pin tox<4 for stable branches testing horizon npm jobs start failing due to some recent changes in tox4. As discussed on open-discuss ML, it is decided to keep testing the stable branch with the tox version that they release with and not with the tox 4. This patch set the 'ensure_tox_version' to use 4, which will tell zuul to install tox<4 if it's not present. For more info. please refer [1]. [1] https://lists.openstack.org/pipermail/openstack-discuss/2023-January/031668.html Change-Id: I3652d87d817030bcd3855d26bfa41b8b7061d08e (cherry picked from commit 4cc580d94b1e579c307583d2f8eb2a5cb699b14f) --- .zuul.d/nodejs-jobs.yaml | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/.zuul.d/nodejs-jobs.yaml b/.zuul.d/nodejs-jobs.yaml index 4273829d93..86586275bc 100644 --- a/.zuul.d/nodejs-jobs.yaml +++ b/.zuul.d/nodejs-jobs.yaml @@ -6,6 +6,10 @@ vars: node_version: 10 tox_constraints_file: "{{ ansible_user_dir }}/{{ zuul.projects['opendev.org/openstack/requirements'].src_dir }}/upper-constraints.txt" + # NOTE: This is stable branch (<=stable/zed) job and new tox 4 + # require some changes in tox.ini to be compatible with it. Let's + # pin tox <4 for stable branches testing (<=stable/zed). + ensure_tox_version: '<4' nodeset: ubuntu-focal pre-run: playbooks/horizon-nodejs/pre.yaml required-projects: @@ -20,6 +24,10 @@ vars: node_version: 10 tox_constraints_file: "{{ ansible_user_dir }}/{{ zuul.projects['opendev.org/openstack/requirements'].src_dir }}/upper-constraints.txt" + # NOTE: This is stable branch (<=stable/zed) job and new tox 4 + # require some changes in tox.ini to be compatible with it. Let's + # pin tox <4 for stable branches testing (<=stable/zed). + ensure_tox_version: '<4' pre-run: playbooks/horizon-nodejs/pre.yaml nodeset: ubuntu-focal required-projects: