Re: [pve-devel] [PATCH installer v2 1/6] fix #4829: install: add new ZFS `arc_max` setup option

[Christoph Heiss <c.heiss@proxmox.com>]

On Tue, Oct 24, 2023 at 08:59:36AM -0300, Gilberto Ferreira via pve-devel wrote:
> Date: Tue, 24 Oct 2023 08:59:36 -0300
> From: Gilberto Ferreira <gilberto.nunes32@gmail.com>
> To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>
> Subject: Re: [pve-devel] [PATCH installer v2 1/6] fix #4829: install: add
>  new ZFS `arc_max` setup option
>
>
> Hi there.
> Now, that's a good option in the installer.
> I wonder if this option will be available post-intall, like some box in the
> ZFS manager from WEB GUI!
> That's would be nice.
>

Currently, this isn't planned, although - since that setting is exposed
after all in the installer in the future - it would be kind of sensible
to add it to the GUI as well, I guess.
But as a separate series from this one, of course.

CC @Thomas - what do you think?

If you think this is something worth doing, I'd create a proper bugzilla
ticket with some more details.

My thinking here (roughly) would be:

* Move the {default,clamp}_zfs_arc_max() stuff to pve-common
* As for the API, probably shove it under `/nodes/{node}/config`?
  It's a per-node configuration after all, it seems to fit there best
  without introducing another endpoint just for this
* GUI-wise; if we do the above, it probably should also go under node ->
  system -> options ..
  Otherwise, add a small, separate panel for options to node -> disks ->
  zfs maybe?