diff --git a/meta/main.yml b/meta/main.yml index fe6a30e..054c844 100644 --- a/meta/main.yml +++ b/meta/main.yml @@ -3,7 +3,7 @@ galaxy_info: description: Deploy Barbican with OpenStack-Ansible company: OpenStack license: Apache-2.0 - min_ansible_version: 1.9 + min_ansible_version: 2.0 platforms: - name: Ubuntu versions: diff --git a/tasks/install.yml b/tasks/install.yml index 7fdca4b..c31098a 100644 --- a/tasks/install.yml +++ b/tasks/install.yml @@ -39,49 +39,19 @@ retries: 5 delay: 2 -- name: Get local venv checksum - stat: - path: "/var/cache/{{ barbican_venv_download_url | basename }}" - get_md5: False - when: - - not barbican_developer_mode | bool - register: local_venv_stat - -- name: Get remote venv checksum - uri: - url: "{{ barbican_venv_download_url | replace('tgz', 'checksum') }}" - return_content: True - when: - - not barbican_developer_mode | bool - register: remote_venv_checksum - -# 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: "{{ barbican_venv_download_url }}" dest: "/var/cache/{{ barbican_venv_download_url | basename }}" - force: yes - failed_when: false - register: get_venv - when: - - not barbican_developer_mode | 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 }}) - -- name: Set barbican get_venv fact - set_fact: - barbican_get_venv: "{{ get_venv }}" + checksum: "sha1:{{ lookup('url', barbican_venv_download_url | replace('tgz', 'checksum')) }}" + register: barbican_get_venv + when: not barbican_developer_mode | bool - name: Remove existing venv file: path: "{{ barbican_bin | dirname }}" state: absent - when: - - barbican_get_venv | changed + when: barbican_get_venv | changed - name: Create barbican venv dir file: @@ -115,8 +85,7 @@ until: install_packages|success retries: 5 delay: 2 - when: - - barbican_get_venv | failed or barbican_developer_mode | bool + when: barbican_developer_mode | bool notify: Restart barbican services - name: Update virtualenv path @@ -124,4 +93,4 @@ virtualenv-tools --update-path=auto {{ barbican_bin | dirname }} when: - not barbican_developer_mode | bool - - barbican_get_venv | success + - barbican_get_venv | changed or barbican_venv_dir | changed