Re: [pve-devel] [PATCH manager 1/2] api: pools: support nested pools

2023-11-17 Thread Fabian Grünbichler
> Wolfgang Bumiller hat am 17.11.2023 12:58 CET > geschrieben: > minor issue > > diff --git a/PVE/API2/Pool.pm b/PVE/API2/Pool.pm > > index 51ac71941..54e744558 100644 > > --- a/PVE/API2/Pool.pm > > +++ b/PVE/API2/Pool.pm > > @@ -354,6 +476,9 @@ __PACKAGE__->register_method ({ > > > >

Re: [pve-devel] [PATCH manager 1/2] api: pools: support nested pools

2023-11-17 Thread Wolfgang Bumiller
minor issue On Fri, Nov 17, 2023 at 08:09:10AM +0100, Fabian Grünbichler wrote: > since poolid can now contain `/`, it's not possible to use it (properly) as > path parameter anymore. > > accordingly: > - merge `read_pool` (`GET /pools/{poolid}`) into 'index' (`GET > /pools/?poolid={poolid}`) (

[pve-devel] [PATCH manager 1/2] api: pools: support nested pools

2023-11-16 Thread Fabian Grünbichler
since poolid can now contain `/`, it's not possible to use it (properly) as path parameter anymore. accordingly: - merge `read_pool` (`GET /pools/{poolid}`) into 'index' (`GET /pools/?poolid={poolid}`) (requires clients to extract the only member of the returned array if they want to query an i