openstack/ansible-role-python_venv_build
Code Issues Proposed changes

Slurp constraints for all hosts

Browse Source 
This fixes bug when content slurp has been performed only for single
host. Others were having invalid contstraints file with single record
in it.

Change-Id: I7189e061f6c8ab3dfb69dda6a9e7f974f8bdc06b
This commit is contained in:
Dmitriy Rabotyagov
2021-03-01 15:23:27 +02:00
parent 90f8fe6801
commit 20f8ad42ce
2 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
tasks/python_venv_install.yml
View File

@@ -13,6 +13,15 @@
# See the License for the specific language governing permissions and # See the License for the specific language governing permissions and
# limitations under the License. # limitations under the License.
- name: Slurp up the constraints file for later re-deployment
delegate_to: "{{ venv_build_host }}"
become: "{{ venv_build_host == 'localhost' }}"
slurp:
src: "{{ venv_build_host_requirements_path }}/{{ venv_install_destination_path | basename }}-constraints.txt"
register: _constraints_file_slurp
run_once: true
when: venv_wheel_build_enable | bool
# TODO(odyssey4me): # TODO(odyssey4me):
# Set a fact for the selective inclusion of the build package list. # Set a fact for the selective inclusion of the build package list.
# Perhaps do this if the distro/architecture of the target host differs # Perhaps do this if the distro/architecture of the target host differs

7
tasks/python_venv_wheel_build.yml
View File

@@ -157,10 +157,3 @@
{{ file_name.split('-')[0] | lower }}=={{ (file_name.split('-')[1].split('_')) | join('.post') | lower }} {{ file_name.split('-')[0] | lower }}=={{ (file_name.split('-')[1].split('_')) | join('.post') | lower }}
{% endfor %} {% endfor %}
dest: "{{ venv_build_host_requirements_path }}/{{ venv_install_destination_path | basename }}-constraints.txt" dest: "{{ venv_build_host_requirements_path }}/{{ venv_install_destination_path | basename }}-constraints.txt"
- name: Slurp up the constraints file for later re-deployment
slurp:
src: "{{ venv_build_host_requirements_path }}/{{ venv_install_destination_path | basename }}-constraints.txt"
register: _constraints_file_slurp
tags:
- install