From: Dimitri Savineau <dsavinea@redhat.com>
Date: Thu, 10 Sep 2020 15:27:37 +0000 (-0400)
Subject: container: run engine/common roles on first client
X-Git-Tag: v6.0.0alpha2~44
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=8ecbdc6ede7e26d053f87acde99986fddb0fe070;p=ceph-ansible.git

container: run engine/common roles on first client

We already do this in the site-container.yml playbook because we don't
need docker/podman installed on all client nodes and having the
container image only on the first client node.

Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>
---

diff --git a/dashboard.yml b/dashboard.yml
index e647a8e63..d772fad53 100644
--- a/dashboard.yml
+++ b/dashboard.yml
@@ -20,10 +20,12 @@
       tags: ['ceph_update_config']
     - import_role:
         name: ceph-container-engine
+      when: (group_names != ['clients']) or (inventory_hostname == groups.get('clients', [''])|first)
     - import_role:
         name: ceph-container-common
         tasks_from: registry
       when:
+        - (group_names != ['clients']) or (inventory_hostname == groups.get('clients', [''])|first)
         - not containerized_deployment | bool
         - ceph_docker_registry_auth | bool
     - import_role:
diff --git a/infrastructure-playbooks/rolling_update.yml b/infrastructure-playbooks/rolling_update.yml
index 5bf8f9f89..5382f4c67 100644
--- a/infrastructure-playbooks/rolling_update.yml
+++ b/infrastructure-playbooks/rolling_update.yml
@@ -88,16 +88,17 @@
     - import_role:
         name: ceph-infra
 
+    - import_role:
+        name: ceph-validate
+
     - import_role:
         name: ceph-container-common
         tasks_from: registry
       when:
+        - (group_names != ['clients']) or (inventory_hostname == groups.get('clients', [''])|first)
         - containerized_deployment | bool
         - ceph_docker_registry_auth | bool
 
-    - import_role:
-        name: ceph-validate
-
     - set_fact: rolling_update=true
 
 - name: upgrade ceph mon cluster
@@ -806,7 +807,9 @@
       when: not containerized_deployment | bool
     - import_role:
         name: ceph-container-common
-      when: containerized_deployment | bool
+      when:
+        - (group_names != ['clients']) or (inventory_hostname == groups.get('clients', [''])|first)
+        - containerized_deployment | bool
     - import_role:
         name: ceph-config
     - import_role: