From: René Højbjerg Larsen <rhl@jfm.dk>
Date: Fri, 3 Mar 2023 14:15:16 +0000 (+0100)
Subject: ceph-mgr: Fix reference to copy_admin_key variable
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=b98a92101b5d0a0754c93094f0720c24b68795f7;p=ceph-ansible.git

ceph-mgr: Fix reference to copy_admin_key variable

Enabling installation of the admin key to mgr nodes by setting
"copy_admin_key: true" is broken. This is because the variable is not
referenced correctly (using inline Jinja2 templating).

Signed-off-by: René Højbjerg Larsen <rhl@jfm.dk>
(cherry picked from commit 09590c0683f3bff21a1b06de7f4378a2f8447e99)
---

diff --git a/roles/ceph-mgr/tasks/common.yml b/roles/ceph-mgr/tasks/common.yml
index 4b59e4783..f9cc63f6e 100644
--- a/roles/ceph-mgr/tasks/common.yml
+++ b/roles/ceph-mgr/tasks/common.yml
@@ -52,7 +52,7 @@
     - name: set_fact _mgr_keys
       set_fact:
         _mgr_keys:
-          - { 'name': 'client.admin', 'path': "/etc/ceph/{{ cluster }}.client.admin.keyring", 'copy_key': copy_admin_key }
+          - { 'name': 'client.admin', 'path': "/etc/ceph/{{ cluster }}.client.admin.keyring", 'copy_key': "{{ copy_admin_key }}" }
           - { 'name': "mgr.{{ ansible_facts['hostname'] }}", 'path': "/var/lib/ceph/mgr/{{ cluster }}-{{ ansible_facts['hostname'] }}/keyring", 'copy_key': true }
 
     - name: get keys from monitors