Merge "Factor out OpenStack release detection playbook"

ansible/bifrost.yml

@@ -1,19 +1,7 @@
 ---
-- name: Detect openstack_release variable
+- import_playbook: detect-release.yml
-  hosts: bifrost
+  vars:
-  gather_facts: false
+    detect_release_hosts: bifrost
-  tasks:
-    - name: Get current kolla-ansible version number
-      local_action: command python -c "import pbr.version; print(pbr.version.VersionInfo('kolla-ansible'))"
-      register: kolla_ansible_version
-      changed_when: false
-      when: openstack_release == "auto"
-
-    - name: Set openstack_release variable
-      set_fact:
-        openstack_release: "{{ kolla_ansible_version.stdout }}"
-      when: openstack_release == "auto"
-  tags: always
 
 - name: Apply role bifrost
   hosts: bifrost

ansible/detect-release.yml

@@ -0,0 +1,16 @@
+---
+- name: Detect openstack_release variable
+  hosts: "{{ detect_release_hosts }}"
+  gather_facts: false
+  tasks:
+    - name: Get current kolla-ansible version number
+      local_action: command python -c "import pbr.version; print(pbr.version.VersionInfo('kolla-ansible'))"
+      register: kolla_ansible_version
+      changed_when: false
+      when: openstack_release == "auto"
+
+    - name: Set openstack_release variable
+      set_fact:
+        openstack_release: "{{ kolla_ansible_version.stdout }}"
+      when: openstack_release == "auto"
+  tags: always

ansible/mariadb_backup.yml

@@ -1,19 +1,7 @@
 ---
-- name: Detect openstack_release variable
+- import_playbook: detect-release.yml
-  hosts: mariadb
+  vars:
-  gather_facts: false
+    detect_release_hosts: mariadb
-  tasks:
-    - name: Get current kolla-ansible version number
-      local_action: command python -c "import pbr.version; print(pbr.version.VersionInfo('kolla-ansible'))"
-      register: kolla_ansible_version
-      changed_when: false
-      when: openstack_release == "auto"
-
-    - name: Set openstack_release variable
-      set_fact:
-        openstack_release: "{{ kolla_ansible_version.stdout }}"
-      when: openstack_release == "auto"
-  tags: always
 
 - name: Backup MariaDB
   hosts: mariadb

ansible/mariadb_recovery.yml

@@ -1,19 +1,7 @@
 ---
-- name: Detect openstack_release variable
+- import_playbook: detect-release.yml
-  hosts: mariadb
+  vars:
-  gather_facts: false
+    detect_release_hosts: mariadb
-  tasks:
-    - name: Get current kolla-ansible version number
-      local_action: command python -c "import pbr.version; print(pbr.version.VersionInfo('kolla-ansible'))"
-      register: kolla_ansible_version
-      changed_when: false
-      when: openstack_release == "auto"
-
-    - name: Set openstack_release variable
-      set_fact:
-        openstack_release: "{{ kolla_ansible_version.stdout }}"
-      when: openstack_release == "auto"
-  tags: always
 
 - name: Recover mariadb
   hosts: mariadb

ansible/site.yml

@@ -32,21 +32,9 @@
         - (ansible_play_batch | length) != (groups['all'] | length)
   tags: always
 
-- name: Detect openstack_release variable
+- import_playbook: detect-release.yml
-  hosts: all
+  vars:
-  gather_facts: false
+    detect_release_hosts: all
-  tasks:
-    - name: Get current kolla-ansible version number
-      local_action: command python -c "import pbr.version; print(pbr.version.VersionInfo('kolla-ansible'))"
-      register: kolla_ansible_version
-      changed_when: false
-      when: openstack_release == "auto"
-
-    - name: Set openstack_release variable
-      set_fact:
-        openstack_release: "{{ kolla_ansible_version.stdout }}"
-      when: openstack_release == "auto"
-  tags: always
 
 - name: Apply role prechecks
   gather_facts: false