public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH installer v3 0/3] expose zfs arc size setting for all products
@ 2024-04-15 13:20 Christoph Heiss
  2024-04-15 13:20 ` [pve-devel] [PATCH installer v3 1/3] tui: NumericEditView: add optional placeholder value Christoph Heiss
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Christoph Heiss @ 2024-04-15 13:20 UTC (permalink / raw)
  To: pve-devel

As suggested by Thomas, leaves the ZFS default if the user never touches
the setting in the installer (i.e. not writing a modprobe file).
See also the discussion in v1 [0].

[0] https://lists.proxmox.com/pipermail/pve-devel/2024-February/061659.html

Testing
-------
Tested the installation of PVE and PBS (PMG is handled the same as PBS),
with each once letting the arc size setting untouched and once setting
it to some specific value. Also checked for each whether the correct
default value was displayed. Afterwards, checked that for PVE the module
parameter was always written to /etc/modprobe.d/, for PBS that it was
only written in case it was explicitly set.

Previous revisions
------------------
v2: https://lists.proxmox.com/pipermail/pve-devel/2024-February/061667.html
v1: https://lists.proxmox.com/pipermail/pve-devel/2023-November/060898.html

Changes v2 -> v3:
  * tui: when empty & focused, do not show the placeholder value at all
  * gui: rework using Gtk3::Adjustment based on Maximilanos suggestion

Changes v1 -> v2:
  * rebased on latest master
  * add placeholder functionality for arc max size in TUI
  * "emulate" placeholder functionality in GTK on best-effort basis

Christoph Heiss (3):
  tui: NumericEditView: add optional placeholder value
  tui: expose arc size setting for zfs bootdisks for all products
  proxinstall: expose arc size setting for zfs bootdisks for all
    products

 Proxmox/Install/RunEnv.pm                   |  3 +-
 proxinstall                                 | 26 ++++-----
 proxmox-installer-common/src/options.rs     |  3 +-
 proxmox-tui-installer/src/views/bootdisk.rs | 45 ++++++++------
 proxmox-tui-installer/src/views/mod.rs      | 65 +++++++++++++++++++--
 5 files changed, 105 insertions(+), 37 deletions(-)

--
2.42.0





^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2024-05-24 10:31 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-15 13:20 [pve-devel] [PATCH installer v3 0/3] expose zfs arc size setting for all products Christoph Heiss
2024-04-15 13:20 ` [pve-devel] [PATCH installer v3 1/3] tui: NumericEditView: add optional placeholder value Christoph Heiss
2024-04-15 13:20 ` [pve-devel] [PATCH installer v3 2/3] tui: expose arc size setting for zfs bootdisks for all products Christoph Heiss
2024-04-15 13:20 ` [pve-devel] [PATCH installer v3 3/3] proxinstall: " Christoph Heiss
2024-05-24 10:30 ` [pve-devel] [PATCH installer v3 0/3] expose zfs arc size setting " Christoph Heiss

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal