[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