[pve-devel] [PATCH manager] gui: add raid warnings for ceph and zfs
Thomas Lamprecht
t.lamprecht at proxmox.com
Fri Jun 21 11:30:13 CEST 2019
Am 6/21/19 um 11:15 AM schrieb Dominik Csapak:
> Wording is from the installer
>
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
> www/manager6/ceph/OSD.js | 6 ++++++
> www/manager6/node/ZFS.js | 6 ++++++
> 2 files changed, 12 insertions(+)
>
> diff --git a/www/manager6/ceph/OSD.js b/www/manager6/ceph/OSD.js
> index db2e1bd9..5d57c398 100644
> --- a/www/manager6/ceph/OSD.js
> +++ b/www/manager6/ceph/OSD.js
> @@ -22,6 +22,12 @@ Ext.define('PVE.CephCreateOsd', {
> method: 'POST',
> items: [
> {
> + xtype: 'displayfield',
> + padding: '0 0 10 0',
> + userCls: 'pve-hint',
> + value: 'Note: Ceph is not compatible with disks backed by a hardware RAID controller. For details see the reference documentation.',
> + },
> + {
> xtype: 'inputpanel',
> onGetValues: function(values) {
> Object.keys(values || {}).forEach(function(name) {
> diff --git a/www/manager6/node/ZFS.js b/www/manager6/node/ZFS.js
> index 7003442e..517fbf56 100644
> --- a/www/manager6/node/ZFS.js
> +++ b/www/manager6/node/ZFS.js
> @@ -41,6 +41,12 @@ Ext.define('PVE.node.CreateZFS', {
> method: 'POST',
> items: [
> {
> + xtype: 'displayfield',
> + padding: '0 0 10 0',
> + userCls: 'pve-hint',
> + value: 'Note: ZFS is not compatible with disks backed by a hardware RAID controller. For details see the reference documentation.',
> + },
> + {
> xtype: 'inputpanel',
> onGetValues: function(values) {
> return values;
>
You're here in the gui with possibility to link to the respective
paragraph on the local installed docs, could be good to do instead of
the "see ref. docs"? I only wrote that because /currently/ we do not
have the docs available in the installer.. :)
More information about the pve-devel
mailing list