[pve-devel] applied: [PATCH manager] make bluestore the default on the api

Wolfgang Bumiller w.bumiller at proxmox.com
Fri Oct 20 12:05:12 CEST 2017


applied

On Fri, Oct 20, 2017 at 11:44:25AM +0200, Dominik Csapak wrote:
> with this we also have to send '0' to from the frontend, when the
> bluestore checkbox is not checked
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  PVE/API2/Ceph.pm         | 6 +++---
>  www/manager6/ceph/OSD.js | 1 +
>  2 files changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/PVE/API2/Ceph.pm b/PVE/API2/Ceph.pm
> index e523ecac..54bad234 100644
> --- a/PVE/API2/Ceph.pm
> +++ b/PVE/API2/Ceph.pm
> @@ -212,9 +212,9 @@ __PACKAGE__->register_method ({
>  		optional => 1,
>  	    },
>  	    bluestore => {
> -		description => "Use bluestore instead of filestore.",
> +		description => "Use bluestore instead of filestore. This is the default.",
>  		type => 'boolean',
> -		default => 0,
> +		default => 1,
>  		optional => 1,
>  	    },
>  	},
> @@ -236,7 +236,7 @@ __PACKAGE__->register_method ({
>  
>  	PVE::CephTools::check_ceph_installed('ceph_osd');
>  
> -	my $bluestore = $param->{bluestore} // 0;
> +	my $bluestore = $param->{bluestore} // 1;
>  
>  	my $journal_dev;
>  	my $wal_dev;
> diff --git a/www/manager6/ceph/OSD.js b/www/manager6/ceph/OSD.js
> index 8e88d071..cb358eab 100644
> --- a/www/manager6/ceph/OSD.js
> +++ b/www/manager6/ceph/OSD.js
> @@ -119,6 +119,7 @@ Ext.define('PVE.CephCreateOsd', {
>  		    xtype: 'pvecheckbox',
>  		    name: 'bluestore',
>  		    fieldLabel: 'Bluestore',
> +		    uncheckedValue: '0',
>  		    value: '1'
>  		}
>              ]
> -- 
> 2.11.0



More information about the pve-devel mailing list