client: do not rely on copy_admin_key to import keys

Relying on `copy_admin_key` to import created keys on client nodes makes
us obliged to copy admin key on those nodes which is not something we might
want.
We should use the fact `condition_copy_admin_key` which will be set to
`True` when the delegated node is a mon which means we can import keys
without taking care of admin keyring.

Fixes: #2867

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
pull/2885/head
Guillaume Abrioux 2018-07-11 16:03:10 +02:00 committed by mergify[bot]
parent a1b3d5b7c3
commit 5ef5fcd0b6
1 changed files with 1 additions and 1 deletions

View File

@ -50,7 +50,7 @@
containerized: "{{ docker_exec_cmd | default('') }}" containerized: "{{ docker_exec_cmd | default('') }}"
cluster: "{{ cluster }}" cluster: "{{ cluster }}"
dest: "{{ ceph_conf_key_directory }}" dest: "{{ ceph_conf_key_directory }}"
import_key: "{{ copy_admin_key }}" import_key: "{{ condition_copy_admin_key }}"
mode: "{{ item.mode|default(omit) }}" mode: "{{ item.mode|default(omit) }}"
with_items: "{{ keys }}" with_items: "{{ keys }}"
delegate_to: "{{ delegated_node }}" delegate_to: "{{ delegated_node }}"