[pve-devel] applied: [PATCH manager 1/5] ceph: make size 3/min 2 default with createpool

[Fabian Grünbichler]

On Wed, Aug 09, 2017 at 11:42:52AM +0200, Dominik Csapak wrote:
> in the gui this is already the default, so make it also the default
> in the backend (also 2/1 is really bad as a default)
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  PVE/API2/Ceph.pm | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/PVE/API2/Ceph.pm b/PVE/API2/Ceph.pm
> index 601f852e..7cf07b70 100644
> --- a/PVE/API2/Ceph.pm
> +++ b/PVE/API2/Ceph.pm
> @@ -1506,7 +1506,7 @@ __PACKAGE__->register_method ({
>  	    size => {
>  		description => 'Number of replicas per object',
>  		type => 'integer',
> -		default => 2,
> +		default => 3,
>  		optional => 1,
>  		minimum => 1,
>  		maximum => 7,
> @@ -1514,7 +1514,7 @@ __PACKAGE__->register_method ({
>  	    min_size => {
>  		description => 'Minimum number of replicas per object',
>  		type => 'integer',
> -		default => 1,
> +		default => 2,
>  		optional => 1,
>  		minimum => 1,
>  		maximum => 7,
> @@ -1546,8 +1546,8 @@ __PACKAGE__->register_method ({
>  	    if ! -f $pve_ckeyring_path;
>  
>  	my $pg_num = $param->{pg_num} || 64;
> -	my $size = $param->{size} || 2;
> -	my $min_size = $param->{min_size} || 1;
> +	my $size = $param->{size} || 3;
> +	my $min_size = $param->{min_size} || 2;
>  	my $rados = PVE::RADOS->new();
>  
>  	$rados->mon_command({
> -- 
> 2.11.0
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel