Always build wheels by default

Due to some corner cases that are possible with currnet logic,
it was decided to simply it and always build wheel regardless of host
count. While runtime might take a bit longer, it's always better for
scaling up to already have wheels prepared.

Closes-Bug: #2004252
Change-Id: I5f53db8476eb394516fb35d593932d2552b95a57
This commit is contained in:
Dmitriy Rabotyagov 2023-02-07 18:34:57 +01:00
parent 7fd75008cf
commit 4715b470f9
3 changed files with 6 additions and 5 deletions

View File

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

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.

View File

@ -77,8 +77,6 @@ _venv_wheels_first_play_hosts: |
{% endfor %}
{{ 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_build_dist_arch: "{{ (ansible_facts['distribution'] | lower) | replace(' ', '_') }}-{{ ansible_facts['distribution_version'].split('.')[:2] | join('.') }}-{{ ansible_facts['architecture'] | lower }}"