Re: [pdm-devel] [PATCH proxmox-api-types v3 2/6] sdn: add list/create vnet endpoints

[Wolfgang Bumiller <w.bumiller@proxmox.com>]

On Wed, Sep 03, 2025 at 02:07:45PM +0200, Stefan Hanreich wrote:
> On 9/3/25 11:36 AM, Shannon Sterz wrote:
> > On Tue Sep 2, 2025 at 4:09 PM CEST, Stefan Hanreich wrote:
> >> Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
> >> ---
> >>  pve-api-types/generate.pl | 7 +++++++
> >>  1 file changed, 7 insertions(+)
> >>
> >> diff --git a/pve-api-types/generate.pl b/pve-api-types/generate.pl
> >> index 6f55fb9..eb8436c 100644
> >> --- a/pve-api-types/generate.pl
> >> +++ b/pve-api-types/generate.pl
> >> @@ -80,6 +80,7 @@ Schema2Rust::register_format('bridge-pair' => { code => 'verifiers::verify_bridg
> >>  Schema2Rust::register_format('pve-task-status-type' => { regex => '^(?i:ok|error|warning|unknown)$' });
> >>
> >>  Schema2Rust::register_format('pve-sdn-zone-id' => { regex => '^[a-z][a-z0-9]*[a-z0-9]$' });
> >> +Schema2Rust::register_format('pve-sdn-vnet-id' => { regex => '^[a-z][a-z0-9]*[a-z0-9]$' });
> > 
> > nit: same as previous patch, couldn't this be `^[a-z][a-z0-9]+$`?
> 
> I just copied this over from SDN, but I don't see any issue with
> shortening it, could adjust it in pve-network while I'm at it as well, I
> guess.

Here as well - pve uses `/i`


_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel