]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: rename ceph_admin_key -> client_key in _calc_client_files
authorAdam King <adking@redhat.com>
Thu, 4 Apr 2024 16:28:45 +0000 (12:28 -0400)
committerAdam King <adking@redhat.com>
Tue, 27 Aug 2024 14:00:31 +0000 (10:00 -0400)
I think this variable name is misleading, as we use this same
logic for all client-keyrings that can be setup, not just
the client.admin keyring

Signed-off-by: Adam King <adking@redhat.com>
(cherry picked from commit 9e2327f0c5f0c103e2db8c7d86ceddd177179df9)

src/pybind/mgr/cephadm/serve.py

index 7bfb3f633eed1f8dbeb50951b20c51e9b8ab0e38..245b163b6208792c682eb7daab8f86c8c8192fca 100644 (file)
@@ -1230,9 +1230,9 @@ class CephadmServe:
                     ceph_conf = (0o644, 0, 0, bytes(config), str(config_digest))
                     client_files[host]['/etc/ceph/ceph.conf'] = ceph_conf
                     client_files[host][f'{cluster_cfg_dir}/ceph.conf'] = ceph_conf
-                    ceph_admin_key = (ks.mode, ks.uid, ks.gid, keyring.encode('utf-8'), digest)
-                    client_files[host][ks.path] = ceph_admin_key
-                    client_files[host][f'{cluster_cfg_dir}/{os.path.basename(ks.path)}'] = ceph_admin_key
+                    client_key = (ks.mode, ks.uid, ks.gid, keyring.encode('utf-8'), digest)
+                    client_files[host][ks.path] = client_key
+                    client_files[host][f'{cluster_cfg_dir}/{os.path.basename(ks.path)}'] = client_key
             except Exception as e:
                 self.log.warning(
                     f'unable to calc client keyring {ks.entity} placement {ks.placement}: {e}')