docker: fix keyrings copied on all nodes

All keyring are getting copied to all nodes.
This commit fixes a leftover from a previous code refactor.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1498583
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

diff --git a/roles/ceph-docker-common/tasks/create_configs.yml b/roles/ceph-docker-common/tasks/create_configs.yml
deleted file mode 100644 (file)
index ed97d53..0000000
--- a/roles/ceph-docker-common/tasks/create_configs.yml
+++ /dev/null
@@ -1 +0,0 @@
----

diff --git a/roles/ceph-docker-common/tasks/fetch_configs.yml b/roles/ceph-docker-common/tasks/fetch_configs.yml
deleted file mode 100644 (file)
index cd186a5..0000000
--- a/roles/ceph-docker-common/tasks/fetch_configs.yml
+++ /dev/null
@@ -1,17 +0,0 @@
----
-- name: include stat_ceph_files.yml
-  include: stat_ceph_files.yml
-
-- name: try to fetch ceph config and keys
-  copy:
-    src: "{{ fetch_directory }}/{{ fsid }}/{{ item.0 }}"
-    dest: "{{ item.0 }}"
-    owner: root
-    group: root
-    mode: 0600
-  changed_when: false
-  with_together:
-    - "{{ ceph_config_keys }}"
-    - "{{ statconfig.results | default([]) }}"
-  when:
-    - item.1.stat.exists == true

diff --git a/roles/ceph-docker-common/tasks/main.yml b/roles/ceph-docker-common/tasks/main.yml
index c27a6791d11899a33e36423f056d6b4ff91e0e30..81e95704b0ab3d17bdfa58483b357dee609f5d3b 100644 (file)
--- a/roles/ceph-docker-common/tasks/main.yml
+++ b/roles/ceph-docker-common/tasks/main.yml
@@ -104,9 +104,6 @@
 - name: include dirs_permissions.yml
   include: dirs_permissions.yml
 
-- name: include create_configs.yml
-  include: create_configs.yml
-
 - name: include selinux.yml
   include: selinux.yml
   when:

diff --git a/roles/ceph-mon/tasks/docker/copy_configs.yml b/roles/ceph-mon/tasks/docker/copy_configs.yml
index 2e04a94cf78d213c4191752564cc7aa14c886503..dad616f6120ba861f5745763356820dd2008f4a6 100644 (file)
--- a/roles/ceph-mon/tasks/docker/copy_configs.yml
+++ b/roles/ceph-mon/tasks/docker/copy_configs.yml
@@ -8,7 +8,6 @@
       - /var/lib/ceph/bootstrap-osd/{{ cluster }}.keyring
       - /var/lib/ceph/bootstrap-rgw/{{ cluster }}.keyring
       - /var/lib/ceph/bootstrap-mds/{{ cluster }}.keyring
-      - /var/lib/ceph/bootstrap-rbd/{{ cluster }}.keyring
 
 - name: register rbd bootstrap key
   set_fact:

diff --git a/roles/ceph-nfs/tasks/pre_requisite_container.yml b/roles/ceph-nfs/tasks/pre_requisite_container.yml
index 2b3e52312a377d9d345b05faa640161068f07cb7..3ee0e94fbcb3d17b809a4daf91049a6d8544f892 100644 (file)
--- a/roles/ceph-nfs/tasks/pre_requisite_container.yml
+++ b/roles/ceph-nfs/tasks/pre_requisite_container.yml
@@ -2,7 +2,6 @@
 - name: set config and keys paths
   set_fact:
     ceph_config_keys:
-      - /etc/ceph/{{ cluster }}.client.admin.keyring
       - /var/lib/ceph/bootstrap-rgw/{{ cluster }}.keyring
 
 - name: stat for config and keys

diff --git a/roles/ceph-osd/tasks/copy_configs.yml b/roles/ceph-osd/tasks/copy_configs.yml
index 93572b75c8c13b189ca29a2a5e42553b94a7a753..616e6d28c34194b6e427a096c7fdd49422e2285c 100644 (file)
--- a/roles/ceph-osd/tasks/copy_configs.yml
+++ b/roles/ceph-osd/tasks/copy_configs.yml
@@ -5,7 +5,7 @@
       - /var/lib/ceph/bootstrap-osd/{{ cluster }}.keyring
 
 - name: wait for ceph.conf and keys
-  local_action: wait_for path={{ fetch_directory }}/{{ fsid }}/{{ item.0 }}
+  local_action: wait_for path={{ fetch_directory }}/{{ fsid }}/{{ item }}
   become: false
   with_items: "{{ ceph_config_keys }}"
 
@@ -20,8 +20,8 @@
 
 - name: try to copy ceph config and keys
   copy:
-    src: "{{ fetch_directory }}/{{ fsid }}/{{ item.0 }}"
-    dest: "{{ item.0 }}"
+    src: "{{ fetch_directory }}/{{ fsid }}/{{ item }}"
+    dest: "{{ item }}"
     owner: root
     group: root
     mode: 0644