]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
ceph-mgr: Fix reference to copy_admin_key variable
authorRené Højbjerg Larsen <rhl@jfm.dk>
Fri, 3 Mar 2023 14:15:16 +0000 (15:15 +0100)
committerGuillaume Abrioux <gabrioux@ibm.com>
Thu, 16 Mar 2023 12:14:07 +0000 (13:14 +0100)
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>
roles/ceph-mgr/tasks/common.yml

index 4b59e4783e5b53debb24aa5d464d184e5eaa0793..f9cc63f6e3859380ba954732a38c7c5ad2f815ed 100644 (file)
@@ -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