From 6f8d69f40392c48503a7edc49e07deeec270ddef Mon Sep 17 00:00:00 2001 From: Martin Kopec Date: Thu, 14 May 2020 15:19:28 +0000 Subject: [PATCH] Fix install-plugins role install-plugins role didn't take into account upper constraints when installing plugins, this review fixes that. Change-Id: I82d81aad7952ce9d03c42187e4ae957043f59397 --- .zuul.yaml | 1 + roles/install-plugins/tasks/main.yaml | 6 +++++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/.zuul.yaml b/.zuul.yaml index bb61a6a4..d7c86796 100644 --- a/.zuul.yaml +++ b/.zuul.yaml @@ -52,6 +52,7 @@ - openstack/devstack - openstack/designate - openstack/designate-tempest-plugin + - openstack/requirements roles: - zuul: openstack/python-tempestconf - zuul: openstack/tempest diff --git a/roles/install-plugins/tasks/main.yaml b/roles/install-plugins/tasks/main.yaml index cf35cf1c..688605b8 100644 --- a/roles/install-plugins/tasks/main.yaml +++ b/roles/install-plugins/tasks/main.yaml @@ -6,7 +6,11 @@ - name: Install plugins become: yes - command: tox -evenv-tempest -- pip install {{ item }} + command: + cmd: > + tox -evenv-tempest -- + pip install {{ item }} + --constraint {{ ansible_user_dir }}/{{ zuul.projects['opendev.org/openstack/requirements'].src_dir }}/upper-constraints.txt with_items: "{{ plugins_paths }}" args: chdir: "{{devstack_base_dir}}/tempest"