Re: [pve-devel] [PATCH pve-manager 1/2] permpathstore: add sdn zones

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

On 05/08/2021 16:58, Alexandre Derumier wrote:
> Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
> ---
>  www/manager6/data/PermPathStore.js | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/www/manager6/data/PermPathStore.js b/www/manager6/data/PermPathStore.js
> index 1dc276b6..cf702c03 100644
> --- a/www/manager6/data/PermPathStore.js
> +++ b/www/manager6/data/PermPathStore.js
> @@ -10,6 +10,7 @@ Ext.define('PVE.data.PermPathStore', {
>  	{ 'value': '/access/realm' },
>  	{ 'value': '/nodes' },
>  	{ 'value': '/pool' },
> +	{ 'value': '/sdn/zones' },

Would need guarding similar to how we hide other SDN stuff for now, else it could be
confusing to some. I.e., adding something like the following stanza in the constructor
between the suspendEvents and resumeEvents calls:

if (PVE.SDNInfo) {
  me.add({ value: '/sdn/zones' });
}



>  	{ 'value': '/storage' },
>  	{ 'value': '/vms' },
>      ],
> @@ -32,6 +33,8 @@ Ext.define('PVE.data.PermPathStore', {
>  		    break;
>  		case 'lxc': path = '/vms/' + record.get('vmid');
>  		    break;
> +		case 'sdn': path = '/sdn/zones/' + record.get('sdn');
> +		    break;
>  		case 'storage': path = '/storage/' + record.get('storage');
>  		    break;
>  		case 'pool': path = '/pool/' + record.get('pool');
>