superseded: [PATCH manager/storage 0/2] fix #3936: add validation for ZFS blocksize

[David Riley <d.riley@proxmox.com>]

superseded by:
https://lore.proxmox.com/pve-devel/20260515075159.45099-1-d.riley@proxmox.com/

On 5/12/26 3:48 PM, David Riley wrote:
> This patch series fixes #3936 [0] by implementing validation for the
> ZFS blocksize parameter in both the backend API and the frontend UI.
>
> Currently, providing an invalid blocksize (e.g., a non-power-of-two or
> a value outside the supported range) does not result in an immediate
> API error and therefore delays the failure to the volume creation.
> This patch series ensures that invalid configurations are caught
> early.
>
> Validation [1][2]:
> * Range: 512 bytes to 16 MiB.
> * Format: Allow positive integers with optional 'k' or 'm' suffix.
> * Constraint: Must be a power of two.
>
> [0] https://bugzilla.proxmox.com/show_bug.cgi?id=3936
> [1] https://openzfs.github.io/openzfs-docs/man/v2.4/7/zfsprops.7.html#volblocksize
> [2] https://openzfs.github.io/openzfs-docs/man/v2.4/7/zfsprops.7.html#recordsize
>
>
> pve-manager:
>
> David Riley (1):
>    fix #3936: ui: storage: add validation for ZFS blocksize
>
>   www/manager6/Utils.js               | 34 +++++++++++++++++++++++++++++
>   www/manager6/storage/ZFSEdit.js     |  1 +
>   www/manager6/storage/ZFSPoolEdit.js |  1 +
>   3 files changed, 36 insertions(+)
>
>
> pve-storage:
>
> David Riley (1):
>    fix #3936: api: add zfs-blocksize format
>
>   src/PVE/Storage/Plugin.pm        | 34 ++++++++++++++++++++++++++++++++
>   src/PVE/Storage/ZFSPoolPlugin.pm |  1 +
>   2 files changed, 35 insertions(+)
>
>
> Summary over all repositories:
>    5 files changed, 71 insertions(+), 0 deletions(-)
>