From: Dominik Csapak <d.csapak@proxmox.com>
Date: Fri, 20 Oct 2017 09:44:25 +0000 (+0200)
Subject: make bluestore the default on the api
X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=d7faccb04469f00e473e2ada1a94542fb1a6a276;p=pve-manager-legacy.git

make bluestore the default on the api

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@proxmox.com>
---

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'
 		}
             ]