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 <[email protected]>
> ---
>  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

_______________________________________________
pve-devel mailing list
[email protected]
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to