On 14.12.20 11:20, Dominic Jäger wrote:
> The API standard_option pve-vmid always checks the format pve-vmid. The format
> pve-vmid requires the vmid to be >=100. This is correct, because IDs 1-99 are
> reserved.
>
> To display this correctly in the API documentation (API viewer, qm man pages),
> we have to set the minimum to 100 in the API standard_option.
>
> Signed-off-by: Dominic Jäger <d.jae...@proxmox.com>
> ---
> https://forum.proxmox.com/threads/qm-importovf-man-page.80632/
IIRC, we use this somewhere, albeit I'm currently not remembering where.
In any case, both Dietmar and I have the feeling that this will break some
things. I'd like to avoid that, maybe you could take another look if you find
some case where values <100 are used for some special handling?
>
> src/PVE/JSONSchema.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm
> index 29ada5b..b4596d3 100644
> --- a/src/PVE/JSONSchema.pm
> +++ b/src/PVE/JSONSchema.pm
> @@ -58,7 +58,7 @@ sub get_standard_option {
> register_standard_option('pve-vmid', {
> description => "The (unique) ID of the VM.",
> type => 'integer', format => 'pve-vmid',
> - minimum => 1
> + minimum => 100,
> });
>
> register_standard_option('pve-node', {
>
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
