From 4268636066b18b39668022f859d0490b00a8d7a2 Mon Sep 17 00:00:00 2001 From: Jesse Pretorius Date: Mon, 3 Jul 2017 11:58:12 +0100 Subject: [PATCH] Hedge upgrade process against no software change Some deployers make use of the new role without implementing a change in the software, resulting in the correct local facts not being deployed. To hedge against this, we add some more conditionals to ensure that the facts are deployed. Change-Id: I1e783c5a791ab66fbb693f672d87cbf092cf8112 --- tasks/glance_install.yml | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/tasks/glance_install.yml b/tasks/glance_install.yml index 2366c848..66ef8da6 100644 --- a/tasks/glance_install.yml +++ b/tasks/glance_install.yml @@ -120,9 +120,13 @@ section: glance option: need_service_restart value: True - when: glance_get_venv | changed or - glance_venv_dir | changed or - install_packages | changed + when: (glance_get_venv | changed) or + (glance_venv_dir | changed) or + (install_packages | changed) or + (ansible_local is not defined) or + ('openstack_ansible' not in ansible_local) or + ('glance' not in ansible_local['openstack_ansible']) or + ('need_service_restart' not in ansible_local['openstack_ansible']['glance']) - name: Record the venv tag deployed ini_file: