[pve-devel] [PATCH V2 pve-manager 3/3] Now pveceph can also createmon with Infernalis and Hammer.

Dietmar Maurer dietmar at proxmox.com
Thu Jun 9 08:18:10 CEST 2016


this one looks wrong to me:

> On June 7, 2016 at 1:24 PM Wolfgang Link <w.link at proxmox.com> wrote:
> 
> 
> Ceph changed user name from root to ceph.
> And for startup systemd is used instead of sysvinit.
> ---
>  PVE/API2/Ceph.pm | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/PVE/API2/Ceph.pm b/PVE/API2/Ceph.pm
> index 58e5b35..4f85860 100644
> --- a/PVE/API2/Ceph.pm
> +++ b/PVE/API2/Ceph.pm
> @@ -841,6 +841,8 @@ __PACKAGE__->register_method ({
>  
>  	my $monaddrhash = {}; 
>  
> +	my $systemd_managed = PVE::CephTools::systemd_managed();
> +
>  	foreach my $section (keys %$cfg) {
>  	    next if $section eq 'global';
>  	    my $d = $cfg->{$section};
> @@ -893,8 +895,12 @@ __PACKAGE__->register_method ({
>  			    "--cap mds 'allow' " .
>  			    "--cap osd 'allow *' " .
>  			    "--cap mon 'allow *'");
> +
> +		run_command("cp $pve_mon_key_path.tmp /etc/ceph/ceph.client.admin.keyring")
> if $systemd_managed;
> +		run_command("chown ceph:ceph /etc/ceph/ceph.client.admin.keyring") if
> $systemd_managed;

why do we copy the monitor keyring into a file called
ceph.client.admin.keyring??
Note: Our ceph.client.admin.keyring is stored in $pve_ckeyring_path 



More information about the pve-devel mailing list