all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH pve-common] JSONSchema: Increase vmid option minimum to 100
@ 2020-12-14 10:20 Dominic Jäger
  2020-12-14 11:52 ` Thomas Lamprecht
  0 siblings, 1 reply; 5+ messages in thread
From: Dominic Jäger @ 2020-12-14 10:20 UTC (permalink / raw)
  To: pve-devel

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.jaeger@proxmox.com>
---
https://forum.proxmox.com/threads/qm-importovf-man-page.80632/

 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', {
-- 
2.20.1




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-12-15  8:55 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-14 10:20 [pve-devel] [PATCH pve-common] JSONSchema: Increase vmid option minimum to 100 Dominic Jäger
2020-12-14 11:52 ` Thomas Lamprecht
2020-12-14 12:33   ` Fabian Grünbichler
2020-12-15  8:11     ` Dominic Jäger
2020-12-15  8:55       ` Thomas Lamprecht

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal