]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
ceph-crash: Only deploy key to targeted hosts
authorGaudenz Steinlin <gaudenz.steinlin@cloudscale.ch>
Mon, 10 Aug 2020 09:38:47 +0000 (11:38 +0200)
committerGuillaume Abrioux <gabrioux@redhat.com>
Mon, 19 Oct 2020 18:20:25 +0000 (20:20 +0200)
The current task installs the ceph-crash key to "most" hosts via
"delegate_to". This key is only used by the ceph-crash daemon and should
just be installed on all hosts targeted by this role. There is no need
for using a delegated task.

Signed-off-by: Gaudenz Steinlin <gaudenz.steinlin@cloudscale.ch>
(cherry picked from commit 68cc93fb18d516a04e288418811787355fb0582e)

roles/ceph-crash/tasks/main.yml

index 01b85fd0181c8d1bf80b8e2dab2f987384b2c7c6..4f814dbf8b13370db318a9e55a495aa93e1580cc 100644 (file)
       command: "{{ hostvars[groups[mon_group_name][0]]['container_exec_cmd'] | default('') }} ceph --cluster {{ cluster }} auth get client.crash"
       register: _crash_keys
       delegate_to: "{{ groups.get(mon_group_name)[0] }}"
+      check_mode: False
+      changed_when: False
       run_once: true
 
-    - name: get a list of node where the keyring should be copied
-      set_fact:
-        list_target_node: "{{ list_target_node | default([]) | union(((groups.get('all') | difference(groups.get(grafana_server_group_name, []) + groups.get(client_group_name, []) + groups.get(nfs_group_name, []) + groups.get(iscsi_gw_group_name, []))) + groups.get(item, [])) | unique) }}"
-      run_once: True
-      with_items:
-        - "{{ mon_group_name if groups.get(mon_group_name, []) | length > 0 else [] }}"
-        - "{{ osd_group_name if groups.get(osd_group_name, []) | length > 0 else [] }}"
-        - "{{ mds_group_name if groups.get(mds_group_name, []) | length > 0 else [] }}"
-        - "{{ rgw_group_name if groups.get(rgw_group_name, []) | length > 0 else [] }}"
-        - "{{ rbdmirror_group_name if groups.get(rbdmirror_group_name, []) | length > 0 else [] }}"
-        - "{{ mgr_group_name if groups.get(mgr_group_name, []) | length > 0 else [] }}"
-
     - name: copy ceph key(s) if needed
       copy:
         dest: "{{ ceph_conf_key_directory }}/{{ cluster }}.client.crash.keyring"
         owner: "{{ ceph_uid if containerized_deployment else 'ceph' }}"
         group: "{{ ceph_uid if containerized_deployment else 'ceph' }}"
         mode: "{{ ceph_keyring_permissions }}"
-      with_items: "{{ list_target_node }}"
-      delegate_to: "{{ item }}"
-      run_once: True
-      when: _crash_keys is not skipped
 
 - name: start ceph-crash daemon
   when: containerized_deployment | bool