main: exclude client nodes from facts gathering when delegate_facts_host

This commit excludes client nodes from facts gathering, they are not
needed and can speed up this task.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 865d2eac9ba81bdb9ecbd841e4b73608648dfae2)

diff --git a/infrastructure-playbooks/rolling_update.yml b/infrastructure-playbooks/rolling_update.yml
index cdd9292adc835a1bd1327e777881879b427aee49..4fc784e6ec71288d3000f7c4ffc2cd1a3f49383a 100644 (file)
--- a/infrastructure-playbooks/rolling_update.yml
+++ b/infrastructure-playbooks/rolling_update.yml
@@ -59,19 +59,20 @@
   gather_facts: False
   vars:
     delegate_facts_host: True
-  tasks:
+  roles:
+    - ceph-defaults
+  post_tasks:
     - debug: msg="gather facts on all Ceph hosts for following reference"
 
     - name: gather facts
       setup:
-      when:
-        - not delegate_facts_host | bool
+      when: not delegate_facts_host | bool or inventory_hostname in groups.get(client_group_name, [])
 
     - name: gather and delegate facts
       setup:
       delegate_to: "{{ item }}"
       delegate_facts: True
-      with_items: "{{ groups['all'] }}"
+      with_items: "{{ groups['all'] | difference(groups.get('clients', [])) }}"
       run_once: true
       when:
         - delegate_facts_host | bool

diff --git a/site-docker.yml.sample b/site-docker.yml.sample
index 7ade7d3dcb1ee433dd778c8199dbfa6958dbdab0..b9299317d59caefe2f2b9c58afee007a0a86b313 100644 (file)
--- a/site-docker.yml.sample
+++ b/site-docker.yml.sample
@@ -24,14 +24,14 @@
   pre_tasks:
     - name: gather facts
       setup:
-      when: not delegate_facts_host | bool
+      when: not delegate_facts_host | bool or inventory_hostname in groups.get(client_group_name, [])
       tags: always
 
     - name: gather and delegate facts
       setup:
       delegate_to: "{{ item }}"
       delegate_facts: True
-      with_items: "{{ groups['all'] }}"
+      with_items: "{{ groups['all'] | difference(groups.get('clients', [])) }}"
       run_once: true
       when: delegate_facts_host | bool
       tags: always

diff --git a/site.yml.sample b/site.yml.sample
index e6d4159dac407255761493fa577f580bac7d740b..900f1875ad2d1f4930aa2a47b3ea5b3830fb1e1e 100644 (file)
--- a/site.yml.sample
+++ b/site.yml.sample
@@ -54,13 +54,13 @@
     - name: gather facts
       setup:
       when:
-        - not delegate_facts_host | bool
+        - not delegate_facts_host | bool or inventory_hostname in groups.get(client_group_name, [])
 
     - name: gather and delegate facts
       setup:
       delegate_to: "{{ item }}"
       delegate_facts: True
-      with_items: "{{ groups['all'] }}"
+      with_items: "{{ groups['all'] | difference(groups.get('clients', [])) }}"
       run_once: true
       when:
         - delegate_facts_host | bool