Make explicit fact gathering timeout configurable

Implicit fact gathering performed at the beginning of a play uses the
DEFAULT_GATHER_TIMEOUT [1] config option. Invoking the setup module
directly does not use this option, instead takes a gather_timeout module
argument.

This change adds a fact_gather_timeout variable that is used when
invoking the setup module directly. The default value is given by the
DEFAULT_GATHER_TIMEOUT config option.

[1]
https://docs.ansible.com/ansible/2.4/config.html#default-gather-timeout

Change-Id: I50bcfc264b61686ffcb1341b737f4354067bd03c

playbooks/roles/bifrost-deploy-nodes-dynamic/defaults/main.yml

@@ -20,3 +20,6 @@ noauth_mode: true
 # Ironic to show the instance in Active state.
 wait_for_node_deploy: false
 wait_timeout: 1800
+
+# Timeout for gathering facts.
+fact_gather_timeout: "{{ lookup('config', 'DEFAULT_GATHER_TIMEOUT', on_missing='skip') | default(omit, true) }}"

playbooks/roles/bifrost-deploy-nodes-dynamic/tasks/main.yml

@@ -19,6 +19,7 @@
 # to be inspected.
 - name: "Obtain setup facts"
   setup:
+    gather_timeout: "{{ fact_gather_timeout }}"
 
 - name: "If in noauth mode, unset authentication parameters."
   set_fact:

playbooks/roles/bifrost-ironic-install/defaults/main.yml

@@ -287,3 +287,6 @@ ironic_inspector:
 # NOTE(hwoarang): openSUSE distros may come with recent pip versions so
 # upgrade only what's necessary
 pip_opts: "{{ ((ansible_os_family | lower == 'suse') or (enable_venv | bool)) | ternary('--upgrade-strategy only-if-needed', '--force-reinstall') }}"
+
+# Timeout for gathering facts.
+fact_gather_timeout: "{{ lookup('config', 'DEFAULT_GATHER_TIMEOUT', on_missing='skip') | default(omit, true) }}"

playbooks/roles/bifrost-ironic-install/tasks/bootstrap.yml

@@ -168,6 +168,7 @@
 # NOTE(pas-ha) needed to e.g. pick up new interfaces after libvirt install
 - name: "Refresh facts"
   setup:
+    gather_timeout: "{{ fact_gather_timeout }}"
 
 - name: "Generate ironic Configuration"
   include: ironic_config.yml

playbooks/roles/ironic-enroll-dynamic/defaults/main.yml

@@ -15,3 +15,6 @@ ipa_kernel_url: "{{ ipa_file_protocol }}://{{ hostvars[inventory_hostname]['ansi
 ipa_ramdisk_url: "{{ ipa_file_protocol }}://{{ hostvars[inventory_hostname]['ansible_' + network_interface | replace('-', '_')]['ipv4']['address'] }}:{{file_url_port}}/ipa.initramfs"
 
 noauth_mode: true
+
+# Timeout for gathering facts.
+fact_gather_timeout: "{{ lookup('config', 'DEFAULT_GATHER_TIMEOUT', on_missing='skip') | default(omit, true) }}"

playbooks/roles/ironic-enroll-dynamic/tasks/main.yml

@@ -14,6 +14,7 @@
 ---
 - name: "Collect facts"
   setup:
+    gather_timeout: "{{ fact_gather_timeout }}"
 
 - name: "If in noauth mode, unset authentication parameters."
   set_fact:

playbooks/roles/ironic-inspect-node/defaults/main.yml

@@ -5,3 +5,6 @@ inspection_wait_timeout: 1800
 inventory_dhcp: false
 inventory_dhcp_static_ip: true
 inventory_dns: false
+
+# Timeout for gathering facts.
+fact_gather_timeout: "{{ lookup('config', 'DEFAULT_GATHER_TIMEOUT', on_missing='skip') | default(omit, true) }}"

playbooks/roles/ironic-inspect-node/tasks/main.yml

@@ -14,6 +14,7 @@
 ---
 - name: "Collect facts"
   setup:
+    gather_timeout: "{{ fact_gather_timeout }}"
 
 - name: "If in noauth mode, unset authentication parameters."
   set_fact: