diff --git a/tasks/cinder_install.yml b/tasks/cinder_install.yml index 6404f96b..2b33bc5f 100644 --- a/tasks/cinder_install.yml +++ b/tasks/cinder_install.yml @@ -51,13 +51,42 @@ - cinder-install - cinder-pip-packages +- name: Get local venv checksum + stat: + path: "/var/cache/{{ cinder_venv_download_url | basename }}" + get_md5: False + when: cinder_venv_enabled | bool + register: local_venv_stat + tags: + - cinder-install + - cinder-pip-packages + +- name: Get remote venv checksum + uri: + url: "{{ cinder_venv_download_url | replace('tgz', 'checksum') }}" + return_content: True + when: cinder_venv_enabled | bool + register: remote_venv_checksum + tags: + - cinder-install + - cinder-pip-packages + +# TODO: When project moves to ansible 2 we can pass this a sha256sum which will: +# a) allow us to remove force: yes +# b) allow the module to calculate the checksum of dest file which would +# result in file being downloaded only if provided and dest sha256sum +# checksums differ - name: Attempt venv download get_url: url: "{{ cinder_venv_download_url }}" dest: "/var/cache/{{ cinder_venv_download_url | basename }}" + force: yes ignore_errors: true register: get_venv - when: cinder_venv_enabled | bool + when: + - cinder_venv_enabled | bool + - (local_venv_stat.stat.exists == False or + {{ local_venv_stat.stat.checksum is defined and local_venv_stat.stat.checksum != remote_venv_checksum.content | trim }}) tags: - cinder-install - cinder-pip-packages @@ -70,13 +99,24 @@ - cinder-install - cinder-pip-packages +- name: Remove existing venv + file: + path: "{{ cinder_venv_bin | dirname }}" + state: absent + when: + - cinder_venv_enabled | bool + - cinder_get_venv | changed + tags: + - cinder-install + - cinder-pip-packages + - name: Create cinder venv dir file: path: "{{ cinder_venv_bin | dirname }}" state: directory when: - cinder_venv_enabled | bool - - cinder_get_venv | success + - cinder_get_venv | changed tags: - cinder-install - cinder-pip-packages @@ -88,7 +128,8 @@ copy: "no" when: - cinder_venv_enabled | bool - - cinder_get_venv | success + - cinder_get_venv | changed + notify: Restart cinder services tags: - cinder-install - cinder-pip-packages @@ -118,6 +159,7 @@ when: - cinder_venv_enabled | bool - cinder_get_venv | failed + notify: Restart cinder services tags: - cinder-install - cinder-pip-packages @@ -133,6 +175,7 @@ delay: 2 with_items: cinder_pip_packages when: not cinder_venv_enabled | bool + notify: Restart cinder services tags: - cinder-install - cinder-pip-packages