Re: [PATCH pve-storage v4 02/17] api: plugins/storage: add initial routes and endpoints

["Max R. Carrara" <m.carrara@proxmox.com>]

On Tue Nov 10, 2026 at 3:35 PM CET, Wolfgang Bumiller wrote:
> On Fri, Nov 06, 2026 at 04:45:38PM +0100, Max R. Carrara wrote:
> > Add the following routes / endpoints:
> >
> > * `GET plugins/storage`
> >   Directory index, listing the direct child path components (so,
> >   just 'plugin' currently).
> >
> > * `GET plugins/storage/plugin`
> >   Lists all installed storage plugins and their metadata.
> >   Currently, this is just an array of objects, with each object
> >   containing the plugin's `type()` and Perl module path.
>
> Exposing perl module paths over the API is not a great idea. Consider
> what this would look like when we add/move over to rust or other plugin
> types.
> API-wise we should only care about the type I think. UI/UX wise, plugins
> could get a "display name", version (plugin side, not checked by us at
> all), author, license fields plugins could fill if they want to,
> alongside a non-overideable "built-in" flag sent by the API?

Yeah that makes sense and sounds good to me; I'll see if I can add those
fields in v5 then, probably as part of `plugindata()`.

Thanks!

>
> >
> > * `GET plugins/storage/plugin/{type}`
> >   Returns the metadata for a single plugin given by `{type}`.
> >
> > Signed-off-by: Max R. Carrara <m.carrara@proxmox.com>
> > ---
> >  src/PVE/API2/Makefile                  |   1 +
> >  src/PVE/API2/Plugins/Makefile          |  18 +++++
> >  src/PVE/API2/Plugins/Storage.pm        |  52 ++++++++++++
> >  src/PVE/API2/Plugins/Storage/Makefile  |  17 ++++
> >  src/PVE/API2/Plugins/Storage/Plugin.pm | 106 +++++++++++++++++++++++++
> >  5 files changed, 194 insertions(+)
> >  create mode 100644 src/PVE/API2/Plugins/Makefile
> >  create mode 100644 src/PVE/API2/Plugins/Storage.pm
> >  create mode 100644 src/PVE/API2/Plugins/Storage/Makefile
> >  create mode 100644 src/PVE/API2/Plugins/Storage/Plugin.pm
> >
>
> 8< snip >8