[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