Re: [PATCH v2 storage 02/15] mapping: add base plugin

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

Am 15.07.26 um 12:44 schrieb Mira Limbeck:
> The inspiration to write it as a plugin on top of the current storage
> plugins mostly came from the PCI and USB mapping plugins we already had.

FWIW, the main two use case and rationale to have the mappings for them are:
1. Allow declaring with devices can be used for a VM to allow migration to
   other nodes.
2. Allow to pass out ACLs to such devices cleanly so that non-root users
   can use them too.

If we want/need to support both points, for storage mappings too, then the
resource mappings are a better fit.
If that's not needed  for all relevant use cases we want to care about, then
a storage mapping is indeed simpler.
Without thinking this through that deeply, the storage mapping case probably
does not needs point 2, as storages themselves already have first-class support
in the permission system. And just for the map itself - without the need for
the mappings ACL infrastructure - they are rather overkill. So a per storage
map is probably the better choice.

btw. Wolfgang is checking out improving and leveraging the oneOf support of
our schema to make the storage plugins use that with a mechanism to expose
relevant (common) properties for external plugins for backward compat while
making the schema basically work like it would then work in rust.
While that might take a bit of time, a new property - like Dietmar's proposed
"map" doesn't makes this really harder to do, so I'd not care to much for it.