[pve-devel] [PATCH manager 2/2] keyring: use ckeyring_path variable in chown cmd

Alwin Antreich a.antreich at proxmox.com
Wed Jul 10 15:35:09 CEST 2019


A fixed path for the ceph.client.admin.keyring was used in the chown
command. This patch uses the ckeyring_path variable instead, to minimize
changes should the path change.

Signed-off-by: Alwin Antreich <a.antreich at proxmox.com>
---
 PVE/Ceph/Tools.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/PVE/Ceph/Tools.pm b/PVE/Ceph/Tools.pm
index d6d21ff2..ff00ba15 100644
--- a/PVE/Ceph/Tools.pm
+++ b/PVE/Ceph/Tools.pm
@@ -244,7 +244,7 @@ sub get_or_create_admin_keyring {
 	# we do not want to overwrite it
 	if (! -f $ckeyring_path) {
 	    run_command("cp $pve_ckeyring_path $ckeyring_path");
-	    run_command("chown ceph:ceph /etc/ceph/ceph.client.admin.keyring");
+	    run_command("chown ceph:ceph $ckeyring_path");
 	}
     }
     return $pve_ckeyring_path;
-- 
2.20.1





More information about the pve-devel mailing list