proxmox-mirrors/pve-manager
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-manager synced 2025-08-14 21:28:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

ui: cephfs: only allow CephFS creation if MDS is configured

Browse Source 
Introduce the mdsCount again, I know remember again why I had it in
v3 of my CephFS series.. Use this to disable the CephFS create button
if we have no MDS configured, as this is a requirement.
Further change the gettext for 'No XY configure' to a format string
so that it can be reused.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>
Tested-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Thomas Lamprecht 2018-11-26 16:56:27 +01:00
parent e337caaf46
commit a9d47b2bb3
1 changed files with 26 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
www/manager6/ceph/FS.js
View File

@ -114,7 +114,7 @@ Ext.define('PVE.NodeCephFSPanel', {
xtype: 'pveNodeCephFSPanel', xtype: 'pveNodeCephFSPanel',
mixins: ['Proxmox.Mixin.CBind'], mixins: ['Proxmox.Mixin.CBind'],
title: gettext('Cluster Administration'), title: gettext('CephFS'),
onlineHelp: 'chapter_pvecm', onlineHelp: 'chapter_pvecm',
border: false, border: false,
@ -125,17 +125,23 @@ Ext.define('PVE.NodeCephFSPanel', {
} }
}, },
viewModel: {
parent: null,
data: {
cephfsConfigured: false,
mdsCount: 0
},
formulas: {
canCreateFS: function(get) {
return (!get('cephfsConfigured') && get('mdsCount') > 0);
}
}
},
items: [ items: [
{ {
xtype: 'grid', xtype: 'grid',
title: gettext('CephFS'), emptyText: Ext.String.format(gettext('No {0} configured.'), 'CephFS'),
viewModel: {
parent: null,
data: {
cephfsConfigured: false
}
},
emptyText: gettext('No CephFS configured.'),
controller: { controller: {
xclass: 'Ext.app.ViewController', xclass: 'Ext.app.ViewController',
@ -190,7 +196,7 @@ Ext.define('PVE.NodeCephFSPanel', {
handler: 'onCreate', handler: 'onCreate',
bind: { bind: {
// only one CephFS per Ceph cluster makes sense for now // only one CephFS per Ceph cluster makes sense for now
disabled: '{cephfsConfigured}' disabled: '{!canCreateFS}'
} }
} }
], ],
@ -218,6 +224,7 @@ Ext.define('PVE.NodeCephFSPanel', {
{ {
xtype: 'grid', xtype: 'grid',
title: gettext('Metadata Servers'), title: gettext('Metadata Servers'),
emptyText: Ext.String.format(gettext('No {0} configured.'), 'MDS'),
controller: { controller: {
xclass: 'Ext.app.ViewController', xclass: 'Ext.app.ViewController',
@ -238,9 +245,17 @@ Ext.define('PVE.NodeCephFSPanel', {
} }
})); }));
Proxmox.Utils.monStoreErrors(view, view.rstore); Proxmox.Utils.monStoreErrors(view, view.rstore);
view.rstore.on('load', this.onLoad, this);
view.on('destroy', view.rstore.stopUpdate); view.on('destroy', view.rstore.stopUpdate);
}, },
onLoad: function(store, records, success) {
var vm = this.getViewModel();
if (!success || !records) {
vm.set('mdsCount', 0);
return;
}
vm.set('mdsCount', records.length);
},
onCreateMDS: function() { onCreateMDS: function() {
var view = this.getView(); var view = this.getView();
view.rstore.stopUpdate(); view.rstore.stopUpdate();