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

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

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?

> 
> * `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