[pve-devel] applied: [PATCH v3 storage 1/2] fix #5071: zfs over iscsi: add 'zfs-base-path' configuration option
Thomas Lamprecht
t.lamprecht at proxmox.com
Tue Jul 15 17:54:53 CEST 2025
On Thu, 05 Jun 2025 13:11:08 +0200, Fiona Ebner wrote:
> Use '/dev/zvol' as a base path for new storages for providers 'iet'
> and 'LIO', because that is what modern distributions use.
>
> This is a breaking change regarding the addition of new storages on
> older distributions, but it's enough to specify the base path '/dev'
> explicitly for setups that require it.
>
> [...]
Mario brought my attention to this series, so given that Christoph already
tested it and it seems relatively straight-forward I rebased on perltidy'd
master and applied, thanks!
FWIW, I shortly pondered about potential implications for allowing users with
Datastore.Allocate permission to pass an arbitrary absolute path here, but
that's really no precedent for storage plugins (e.g., DirPlugin), so I do not
think that warrants any special handling here.
[1/2] fix #5071: zfs over iscsi: add 'zfs-base-path' configuration option
commit: d181d0b1eea1ba77a3cbf970d0e076e916ea676f
[2/2] zfs over iscsi: on-add hook: dynamically determine base path
commit: 4fb733a9ac870348511e0d9da5bf90e3278a651a
More information about the pve-devel
mailing list