openinfra/python-tempestconf
Code Issues Proposed changes

Fix install-plugins role

Browse Source 
install-plugins role didn't take into account upper
constraints when installing plugins, this review
fixes that.

Change-Id: I82d81aad7952ce9d03c42187e4ae957043f59397
This commit is contained in:
Martin Kopec 2020-05-14 15:19:28 +00:00 committed by Andreas Jaeger
parent c8a084937e
commit 6f8d69f403
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.zuul.yaml
View File

@ -52,6 +52,7 @@
- openstack/devstack - openstack/devstack
- openstack/designate - openstack/designate
- openstack/designate-tempest-plugin - openstack/designate-tempest-plugin
- openstack/requirements
roles: roles:
- zuul: openstack/python-tempestconf - zuul: openstack/python-tempestconf
- zuul: openstack/tempest - zuul: openstack/tempest

6
roles/install-plugins/tasks/main.yaml
View File

@ -6,7 +6,11 @@
- name: Install plugins - name: Install plugins
become: yes 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 }}" with_items: "{{ plugins_paths }}"
args: args:
chdir: "{{devstack_base_dir}}/tempest" chdir: "{{devstack_base_dir}}/tempest"