[pve-devel] applied: [PATCH storage 2/2] plugin schema: improve description of 'nodes' property

Fabian Grünbichler f.gruenbichler at proxmox.com
Tue Jan 30 10:38:56 CET 2024


applied this on already, thanks!

On October 4, 2023 9:35 am, Fiona Ebner wrote:
> The default description doesn't make it clear what the property is
> used for in the context of storages.
> 
> Signed-off-by: Fiona Ebner <f.ebner at proxmox.com>
> ---
>  src/PVE/Storage/Plugin.pm | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/src/PVE/Storage/Plugin.pm b/src/PVE/Storage/Plugin.pm
> index dc89d13..953b785 100644
> --- a/src/PVE/Storage/Plugin.pm
> +++ b/src/PVE/Storage/Plugin.pm
> @@ -138,7 +138,10 @@ my $defaultData = {
>  	type => { description => "Storage type." },
>  	storage => get_standard_option('pve-storage-id',
>  	    { completion => \&PVE::Storage::complete_storage }),
> -	nodes => get_standard_option('pve-node-list', { optional => 1 }),
> +	nodes => get_standard_option('pve-node-list', {
> +	    description => "List of nodes for which the storage configuration applies.",
> +	    optional => 1,
> +	}),
>  	content => {
>  	    description => "Allowed content types.\n\nNOTE: the value " .
>  		"'rootdir' is used for Containers, and value 'images' for VMs.\n",
> -- 
> 2.39.2
> 
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
> 
> 




More information about the pve-devel mailing list