Re-deploy the Nova venv if it mismatches the repo
We currently have two issues with venvs: - if you update your venv on the repo server, it is not possible for that updated venv to land on the service's container as the get_url task always skips if the file exists (even if the file is different) - if you have an updated venv on the repo server and forcefully delete the cached venv tarball on the service's container, the new tarball will get unarchived over top of the existing venv This commit does the following: - gets the checksum of the /var/cache tarball and downloads checksum file from repo server - updates "Attempt venv download" to only download the venv if the cache doesn't exist or if the local and remote checksums differ - adds a "force: true" to "Attempt venv download" task so that the venv tarball will get re-downloaded when the when condition is true (this is necessary otherwise the download will get skipped since the destination already exists) - adds a new task "Remove existing venv" so we can first remove the venv before we unarchive the potentially new venv from the repo server - updates "Create nova venv dir" and "Unarchive pre-built venv" tasks to only proceed if "nova_get_venv | changed", which prevents these tasks from running when they the venv tarball hasn't changed - adds multiple service restarts to os_nova/tasks/nova_install.yml so that nova will restart correctly should the venv/packages update without any associated config changes NOTE: The reason why we compare local and remote checksum is to avoid unnecessarily downloading the venv when the checksums are in fact the same. On small deploys this is more or less a non-issue but if a deploy w/ thousands of compute nodes re-runs playbooks we want to limit the venv downloads when it's unnecessary. Change-Id: Ibe25126ee38f51fb71f877a54d32dd56d2a8a74d
This commit is contained in:
parent
8a8ad448fe
commit
9b073b6594
@ -296,6 +296,7 @@ nova_requires_pip_packages:
|
||||
- virtualenv
|
||||
- virtualenv-tools
|
||||
- python-keystoneclient # Keystoneclient needed to OSA keystone lib
|
||||
- httplib2
|
||||
|
||||
nova_compute_pip_packages:
|
||||
- libvirt-python
|
||||
|
@ -52,13 +52,42 @@
|
||||
- nova-install
|
||||
- nova-pip-packages
|
||||
|
||||
- name: Get local venv checksum
|
||||
stat:
|
||||
path: "/var/cache/{{ nova_venv_download_url | basename }}"
|
||||
get_md5: False
|
||||
when: nova_venv_enabled | bool
|
||||
register: local_venv_stat
|
||||
tags:
|
||||
- nova-install
|
||||
- nova-pip-packages
|
||||
|
||||
- name: Get remote venv checksum
|
||||
uri:
|
||||
url: "{{ nova_venv_download_url | replace('tgz', 'checksum') }}"
|
||||
return_content: True
|
||||
when: nova_venv_enabled | bool
|
||||
register: remote_venv_checksum
|
||||
tags:
|
||||
- nova-install
|
||||
- nova-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: "{{ nova_venv_download_url }}"
|
||||
dest: "/var/cache/{{ nova_venv_download_url | basename }}"
|
||||
force: yes
|
||||
ignore_errors: true
|
||||
register: get_venv
|
||||
when: nova_venv_enabled | bool
|
||||
when:
|
||||
- nova_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:
|
||||
- nova-install
|
||||
- nova-pip-packages
|
||||
@ -71,13 +100,24 @@
|
||||
- nova-install
|
||||
- nova-pip-packages
|
||||
|
||||
- name: Remove existing venv
|
||||
file:
|
||||
path: "{{ nova_venv_bin | dirname }}"
|
||||
state: absent
|
||||
when:
|
||||
- nova_venv_enabled | bool
|
||||
- nova_get_venv | changed
|
||||
tags:
|
||||
- nova-install
|
||||
- nova-pip-packages
|
||||
|
||||
- name: Create nova venv dir
|
||||
file:
|
||||
path: "{{ nova_venv_bin | dirname }}"
|
||||
state: directory
|
||||
when:
|
||||
- nova_venv_enabled | bool
|
||||
- nova_get_venv | success
|
||||
- nova_get_venv | changed
|
||||
tags:
|
||||
- nova-install
|
||||
- nova-pip-packages
|
||||
@ -89,7 +129,8 @@
|
||||
copy: "no"
|
||||
when:
|
||||
- nova_venv_enabled | bool
|
||||
- nova_get_venv | success
|
||||
- nova_get_venv | changed
|
||||
notify: Restart nova services
|
||||
tags:
|
||||
- nova-install
|
||||
- nova-pip-packages
|
||||
@ -120,6 +161,7 @@
|
||||
when:
|
||||
- nova_venv_enabled | bool
|
||||
- nova_get_venv | failed
|
||||
notify: Restart nova services
|
||||
tags:
|
||||
- nova-install
|
||||
- nova-pip-packages
|
||||
@ -136,6 +178,7 @@
|
||||
with_items:
|
||||
- "{{ nova_pip_packages }}"
|
||||
when: not nova_venv_enabled | bool
|
||||
notify: Restart nova services
|
||||
tags:
|
||||
- nova-install
|
||||
- nova-pip-packages
|
||||
|
Loading…
Reference in New Issue
Block a user