Merge "Move etcd precheck into its own role"

ansible/roles/etcd/tasks/precheck.yml

 ---
+- name: Get container facts
+  kolla_container_facts:
+    name:
+      - etcd
+  register: container_facts
+
+- name: Checking free port for Etcd Peer
+  wait_for:
+    host: "{{ hostvars[inventory_hostname]['ansible_' + api_interface]['ipv4']['address'] }}"
+    port: "{{ etcd_peer_port }}"
+    connect_timeout: 1
+    state: stopped
+  when:
+    - container_facts['etcd'] is not defined
+    - inventory_hostname in groups['etcd']
+
+- name: Checking free port for Etcd Client
+  wait_for:
+    host: "{{ hostvars[inventory_hostname]['ansible_' + api_interface]['ipv4']['address'] }}"
+    port: "{{ etcd_client_port }}"
+    connect_timeout: 1
+    state: stopped
+  when:
+    - container_facts['etcd'] is not defined
+    - inventory_hostname in groups['etcd']

ansible/roles/prechecks/tasks/port_checks.yml

@@ -33,26 +33,6 @@
                 kolla_internal_vip_address | ipaddr(ip_addr_output.stdout.split()[3]) is none"
   when: enable_haproxy | bool
 
-- name: Checking free port for Etcd Peer
-  wait_for:
-    host: "{{ hostvars[inventory_hostname]['ansible_' + api_interface]['ipv4']['address'] }}"
-    port: "{{ etcd_peer_port }}"
-    connect_timeout: 1
-    state: stopped
-  when:
-    - enable_etcd | bool
-    - inventory_hostname in groups['etcd']
-
-- name: Checking free port for Etcd Client
-  wait_for:
-    host: "{{ hostvars[inventory_hostname]['ansible_' + api_interface]['ipv4']['address'] }}"
-    port: "{{ etcd_client_port }}"
-    connect_timeout: 1
-    state: stopped
-  when:
-    - enable_etcd | bool
-    - inventory_hostname in groups['etcd']
-
 - name: Checking free port for Kuryr
   wait_for:
     host: "{{ hostvars[inventory_hostname]['ansible_' + api_interface]['ipv4']['address'] }}"