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

Dominik Csapak d.csapak at proxmox.com
Fri Oct 20 11:44:25 CEST 2017


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