Merge "Define local facts separately only for distro"

tasks/nova_install.yml

@@ -37,6 +37,8 @@
   delay: 2
   tags:
     - nova-pip-packages
+  notify:
+    - "Restart nova services"
 
 - name: Install the python venv
   import_role:
@@ -123,21 +125,31 @@
       condition: "{{ serial_proxy_enabled.rc is defined and serial_proxy_enabled.rc == 0 }}"
       type: "serialconsole"
 
-- name: Initialise the upgrade facts
+- name: Record local facts for distro path
-  ini_file:
+  when: nova_install_method == 'distro'
-    dest: "/etc/ansible/facts.d/openstack_ansible.fact"
+  block:
-    section: nova
+    - name: Record the osa version deployed
-    option: "{{ item.name }}"
+      ini_file:
-    value: "{{ item.state }}"
+        dest: "/etc/ansible/facts.d/openstack_ansible.fact"
-  with_items:
+        section: nova
-    - name: "need_service_restart"
+        option: venv_tag
-      state: "True"
+        value: "{{ nova_venv_tag }}"
-    - name: "need_online_data_migrations"
+
-      state: "True"
+    - name: Initialise the upgrade facts
-  when:
+      ini_file:
-    - (install_packages is changed) or
+        dest: "/etc/ansible/facts.d/openstack_ansible.fact"
-      (ansible_local is not defined) or
+        section: nova
-      ('openstack_ansible' not in ansible_local) or
+        option: "{{ item.name }}"
-      ('nova' not in ansible_local['openstack_ansible']) or
+        value: "{{ item.state }}"
-      ('need_online_data_migrations' not in ansible_local['openstack_ansible']['nova']) or
+      with_items:
-      ('need_service_restart' not in ansible_local['openstack_ansible']['nova'])
+        - name: "need_service_restart"
+          state: "True"
+        - name: "need_online_data_migrations"
+          state: "True"
+      when:
+        - (install_packages is changed) or
+          (ansible_local is not defined) or
+          ('openstack_ansible' not in ansible_local) or
+          ('nova' not in ansible_local['openstack_ansible']) or
+          ('need_online_data_migrations' not in ansible_local['openstack_ansible']['nova']) or
+          ('need_service_restart' not in ansible_local['openstack_ansible']['nova'])