[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