[pve-devel] [PATCH manager v2] fix #4166: restore: add resource pool selector

Thomas Lamprecht t.lamprecht at proxmox.com
Wed Jun 25 16:02:26 CEST 2025


Am 25.06.25 um 15:48 schrieb Maximiliano Sandoval:
> It could be placed there, however I think it makes sense to put it next
> to the storage selector (which could also be argued is an override). I
> could add a `emptyText` with value 'From backup configuration' similar
> as the storage selector though.

My conclusion would then be to move the storage to the rest of the
override settings rather than making the UX pattern even worse here...

But for one that would be certainly not the job of this patch and for
another is the storage a core parameter that is basically always
present (and often explicitly set, as one cannot see what the value
will be, a UX problem but nonetheless the status quo) whereas resource
pools are certainly more of an edge case.

Just like a hypothetical per-volume storage override–or any other
property–should also go in the "Override Setting" section.




More information about the pve-devel mailing list