openstack/openstack-ansible-os_designate
Code Issues Proposed changes

Optimise pip install tasks

Browse Source 
Unlike the Ansible apt module, the Ansible pip module does not
recognise a with_items list and process all the items at once.

To optimise the pip install tasks, this patch replaces the use
of with_items with a join filter so that the pip install task
does an install with all the packages in a list, ensuring that
the execution is one action instead of many.

Change-Id: I4ce24c9d557d173a452ee74a619c771b2bed5af8
This commit is contained in:
Jesse Pretorius 2016-07-18 15:54:16 +01:00
parent 3dadd78703
commit d815a1f485
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tasks/designate_install.yml
View File

@ -51,14 +51,13 @@
- name: Install requires pip packages - name: Install requires pip packages
pip: pip:
name: "{{ item }}" name: "{{ designate_requires_pip_packages | join(' ') }}"
state: latest state: latest
extra_args: "{{ pip_install_options_fact }}" extra_args: "{{ pip_install_options_fact }}"
register: install_packages register: install_packages
until: install_packages|success until: install_packages|success
retries: 5 retries: 5
delay: 2 delay: 2
with_items: "{{ designate_requires_pip_packages }}"
- name: Get local venv checksum - name: Get local venv checksum
stat: stat:
@ -126,7 +125,7 @@
- name: Install pip packages - name: Install pip packages
pip: pip:
name: "{{ item }}" name: "{{ designate_pip_packages | join(' ') }}"
state: latest state: latest
virtualenv: "{{ designate_bin | dirname }}" virtualenv: "{{ designate_bin | dirname }}"
virtualenv_site_packages: "no" virtualenv_site_packages: "no"
@ -135,7 +134,6 @@
until: install_packages|success until: install_packages|success
retries: 5 retries: 5
delay: 2 delay: 2
with_items: "{{ designate_pip_packages }}"
when: when:
- designate_get_venv | failed or designate_developer_mode | bool - designate_get_venv | failed or designate_developer_mode | bool
notify: notify: