openstack/ansible-role-python_venv_build
Code Issues Proposed changes

Add missing 'when' for conditionals

Browse Source
This commit is contained in:
Jesse Pretorius 2018-03-23 12:51:43 +00:00
parent b7010f8e7c
commit 1eb3ac2ba2
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
tasks/python_venv_install.yml
View File

@ -18,12 +18,14 @@
src: "{{ venv_reuse_download_path }}/{{ venv_destination_path | basename }}.checksum" src: "{{ venv_reuse_download_path }}/{{ venv_destination_path | basename }}.checksum"
dest: "{{ venv_destination_path | dirname }}" dest: "{{ venv_destination_path | dirname }}"
register: _venv_checksum_copy register: _venv_checksum_copy
when:
- _src_venv_present.stat.exists | bool - _src_venv_present.stat.exists | bool
- name: Remove existing venv on target host if it is changing - name: Remove existing venv on target host if it is changing
file: file:
path: "{{ venv_destination_path }}" path: "{{ venv_destination_path }}"
state: absent state: absent
when:
- _venv_checksum_copy is mapping - _venv_checksum_copy is mapping
- _venv_checksum_copy | changed - _venv_checksum_copy | changed
@ -37,6 +39,7 @@
src: "{{ venv_reuse_download_path }}/{{ venv_destination_path | basename }}.tgz" src: "{{ venv_reuse_download_path }}/{{ venv_destination_path | basename }}.tgz"
dest: "{{ venv_destination_path }}" dest: "{{ venv_destination_path }}"
remote_src: no remote_src: no
when:
- _venv_checksum_copy is mapping - _venv_checksum_copy is mapping
- _venv_checksum_copy | changed - _venv_checksum_copy | changed
notify: notify: