openstack/ansible-role-python_venv_build
Code Issues Proposed changes

Merge "Always build wheels by default"

Browse Source
This commit is contained in:
Zuul 2023-02-22 02:47:45 +00:00 committed by Gerrit Code Review
commit 84f0a74e77
3 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
defaults/main.yml
View File

@ -122,9 +122,7 @@ venv_wheels_rebuild: no
# If the package concerned is built from a tarball, rather # If the package concerned is built from a tarball, rather
# than from a git source or pypi, then this may be best to # than from a git source or pypi, then this may be best to
# set to false. # set to false.
# Default: Enabled when more then 1 host with same arch/distro venv_wheel_build_enable: True
# is in play
venv_wheel_build_enable: "{{ _venv_wheel_build_enable }}"
# Set the host where the wheels will be built. # Set the host where the wheels will be built.
# If this host is not the same as the target host, then # If this host is not the same as the target host, then

5
releasenotes/notes/wheel_build_enable-ea0bd0cb676128dd.yaml Normal file
View File

@ -0,0 +1,5 @@
---
other:
- |
Default value of ``venv_wheel_build_enable`` has been changed to True.
Previously it was depending on amount of hosts targeted for the play.

2
vars/main.yml
View File

@ -77,8 +77,6 @@ _venv_wheels_first_play_hosts: |
{% endfor %} {% endfor %}
{{ first_hosts }} {{ first_hosts }}
_venv_wheel_build_enable: "{{ (_venv_wheels_play_hosts[ansible_facts['distribution_version'] ~ '_' ~ ansible_facts['architecture']] | length > 1) | bool }}"
_venv_pip_packages: "{{ (venv_default_pip_packages | union(venv_pip_packages)) | sort | select | list }}" _venv_pip_packages: "{{ (venv_default_pip_packages | union(venv_pip_packages)) | sort | select | list }}"
_venv_build_dist_arch: "{{ (ansible_facts['distribution'] | lower) | replace(' ', '_') }}-{{ ansible_facts['distribution_version'].split('.')[:2] | join('.') }}-{{ ansible_facts['architecture'] | lower }}" _venv_build_dist_arch: "{{ (ansible_facts['distribution'] | lower) | replace(' ', '_') }}-{{ ansible_facts['distribution_version'].split('.')[:2] | join('.') }}-{{ ansible_facts['architecture'] | lower }}"