Am 08.09.25 um 20:01 schrieb Max R. Carrara: > GUI Support for Custom Storage Plugins > ====================================== > > tl;dr: > > Add an API method to PVE::Storage::Plugin that returns the definition > for the form view of custom storage plugins. This definition is used by > the frontend to build the form view for creating / editing the storage > config entry of the plugin. The ultimate goal here is that custom > storage plugin devs don't have to (and also *must not*) ever touch > JavaScript to make their plugins show up in the GUI.
Did not check the changes in full, but I see no mentioning of the ACME DNS plugins, which already provide such a mechanism, albeit a bit lower level with no real control how the fields get layouted. Did you check that out and maybe already re-used the underlying parts of the implementation but just not referenced it, or is this something completely new/custom? Further, FWICT this seems to add some parallel infra and need of definitions, might we get away with "just" annotating the existing schemas a bit better, i.e. add the flags for signaling secrets and potentially some layout hints, as the core importance is to be able to input required data to configure a storage, not that it needs to look "perfect", at least not in the initial version of this implementation, as adding more features and complexer layouting later on, especially once we got feedback from integrators and see how they use it. As it's much easier to see what's really required and what might be rather a (not much used) headache to maintain support for in ExtJS and a future Yew based UI. I.e., I'd be totally fine if the initial version would basically look like what the ACME DNS plugin UI does, focusing purely on required parameters, data types and validation over layout. Btw. this is partially related to [0] and I thought there was another issue tracking this, but couldn't find one from a quick search. [0]: https://bugzilla.proxmox.com/show_bug.cgi?id=3420 _______________________________________________ pve-devel mailing list [email protected] https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
