Re: [pve-devel] [PATCH pve-installer 1/6] auto: add early answer file sanity check for RAID configurations

2025-04-28 Thread Christoph Heiss
Thanks for tackling this! On Tue Apr 22, 2025 at 6:27 PM CEST, Michael Köppl wrote: > [..] > +pub fn verify_disks_settings(answer: &Answer) -> Result<()> { > +if let DiskSelection::Selection(selection) = > &answer.disks.disk_selection { > +let min_disks = answer.disks.fs_type.get_min_

[pve-devel] [PATCH kernel] debian: find firmware: quote regex metacharacters in directory name

2025-04-28 Thread Friedrich Weber
The directory name contains the version which may contain characters with semantics as regex metacharacters, e.g. `+` when building a test kernel. If they are not quoted, the script does not remove the directory name from its output, which will cause a build failure in `fwcheck` even though the fir

Re: [pve-devel] [PATCH ha-manager 11/15] test: ha tester: add test cases for strict negative colocation rules

2025-04-28 Thread Fiona Ebner
Am 25.03.25 um 16:12 schrieb Daniel Kral: > Add test cases for strict negative colocation rules, i.e. where services > must be kept on separate nodes. These verify the behavior of the > services in strict negative colocation rules in case of a failover of > the node of one or more of these services

Re: [pve-devel] [PATCH ha-manager 10/15] sim: resources: add option to limit start and migrate tries to node

2025-04-28 Thread Fiona Ebner
Am 25.03.25 um 16:12 schrieb Daniel Kral: > Add an option to the VirtFail's name to allow the start and migrate fail > counts to only apply on a certain node number with a specific naming > scheme. > > This allows a slightly more elaborate test type, e.g. where a service > can start on one node (o

Re: [pve-devel] [PATCH ha-manager 12/15] test: ha tester: add test cases for strict positive colocation rules

2025-04-28 Thread Fiona Ebner
Am 25.03.25 um 16:12 schrieb Daniel Kral: > Add test cases for strict positive colocation rules, i.e. where services > must be kept on the same node together. These verify the behavior of the > services in strict positive colocation rules in case of a failover of > their assigned nodes in the follo

[pve-devel] [PATCH pve_flutter_frontend] navigation: replace deprecated `onPopInvoked` callback

2025-04-28 Thread Shan Shaji
The `onPopInvoked` callback is deprecated as of Flutter v3.22.0-12.0.pre. This patch replaces it with the `onPopInvokedWithResult` callback. Signed-off-by: Shan Shaji --- lib/pages/main_layout_slim.dart | 2 +- lib/states/pve_node_overview_state.dart | 2 -- 2 files changed, 1 insertion

Re: [pve-devel] [PATCH pve-installer 2/6] common: use get_min_disks as single source of truth for RAID config checks

2025-04-28 Thread Christoph Heiss
On Tue Apr 22, 2025 at 6:27 PM CEST, Michael Köppl wrote: > To avoid having multiple places where the minimum number of disks > required for a certain RAID setup is defined, use get_min_disks in the > common installer RAID config checks as well, keeping the behavior the > same as for the answer fil

Re: [pve-devel] [RFC PATCH pve-installer 3/6] close #5887: add sanity check for LVM swapsize and maxroot

2025-04-28 Thread Christoph Heiss
On Tue Apr 22, 2025 at 6:27 PM CEST, Michael Köppl wrote: > Check that the configured swapsize is not greater than the total size > of the disk and that maxroot is at most hdsize / 4. Define the > behavior for the auto-installer as well as the TUI and GUI installers. > > Signed-off-by: Michael Köpp

Re: [pve-devel] [PATCH pve-installer 5/6] common: add more descriptive errors for invalid network configs

2025-04-28 Thread Christoph Heiss
On Tue Apr 22, 2025 at 6:27 PM CEST, Michael Köppl wrote: > Signed-off-by: Michael Köppl > --- > proxmox-installer-common/src/utils.rs | 16 +++- > 1 file changed, 15 insertions(+), 1 deletion(-) > > diff --git a/proxmox-installer-common/src/utils.rs > b/proxmox-installer-common/src/

Re: [pve-devel] [PATCH installer 0/6] add early disk and network sanity checks

2025-04-28 Thread Christoph Heiss
Thanks for tackling this. Reviewed the series and left some comments on the individual patches, mostly some thoughts and nits. Also tested `proxmox-auto-install-assistant validate-answer` for the new cases, as well as the early RAID check in the GUI and the auto installer. On Tue Apr 22, 2025 at

Re: [pve-devel] [PATCH ha-manager 09/15] manager: apply colocation rules when selecting service nodes

2025-04-28 Thread Fiona Ebner
Am 25.03.25 um 16:12 schrieb Daniel Kral: > Add a mechanism to the node selection subroutine, which enforces the > colocation rules defined in the rules config. > > The algorithm manipulates the set of nodes directly, which the service > is allowed to run on, depending on the type and strictness o

Re: [pve-devel] [PATCH ha-manager 09/15] manager: apply colocation rules when selecting service nodes

2025-04-28 Thread Fiona Ebner
Am 11.04.25 um 17:56 schrieb Daniel Kral: > On 4/3/25 14:17, Fabian Grünbichler wrote: >> On March 25, 2025 4:12 pm, Daniel Kral wrote: >>> +sub apply_positive_colocation_rules { >>> +    my ($together, $allowed_nodes) = @_; >>> + >>> +    return if scalar(keys %$together) < 1; >>> + >>> +    my $m

Re: [pve-devel] [PATCH ha-manager 08/15] manager: factor out prioritized nodes in select_service_node

2025-04-28 Thread Fiona Ebner
Am 25.03.25 um 16:12 schrieb Daniel Kral: > Factor out the prioritized node hash set in the select_service_node as > it is used multiple times and makes the intent a little clearer. > > Signed-off-by: Daniel Kral Reviewed-by: Fiona Ebner ___ pve-dev

Re: [pve-devel] [RFC PATCH pve-installer 6/6] closes #5757: common: add checks for valid IPv4 address within subnet

2025-04-28 Thread Christoph Heiss
On Tue Apr 22, 2025 at 6:27 PM CEST, Michael Köppl wrote: > Implement check if the address entered by the user is valid within the > given subnet, i.e. not a network address or broadcast address. > > Partially closes [0]. > > [0] https://bugzilla.proxmox.com/show_bug.cgi?id=5757 > > Signed-off-by:

[pve-devel] [PATCH qemu-server] virtiofs: prevent issue with Windows OS and too many files

2025-04-28 Thread Fiona Ebner
As reported in the community forum [0] and the virtio-win project [1], virtiofsd will run into its open file limit when used with a Windows guest that reads too many files. It's also reported that the issue does not occur with Linux guests and a workaround is using '--inode-file-handles=mandatory'

[pve-devel] Consideration of contributing Apparmor for KVM

2025-04-28 Thread Sven Springer via pve-devel
--- Begin Message --- Hello, for a project we are working on a simple Apparmor profile for KVM-based VMs in Proxmox. For now it's a POC with a static profile for the qemu-system-x86_64 binary. The next step would be to patch the Proxmox Perl code to implement a basic version of dynamic profile

Re: [pve-devel] [PATCH pve-installer 1/6] auto: add early answer file sanity check for RAID configurations

2025-04-28 Thread Michael Köppl
On 4/28/25 13:25, Christoph Heiss wrote: On Tue Apr 22, 2025 at 6:27 PM CEST, Michael Köppl wrote: [..] +pub fn verify_disks_settings(answer: &Answer) -> Result<()> { +if let DiskSelection::Selection(selection) = &answer.disks.disk_selection { +let min_disks = answer.disks.fs_type.ge

Re: [pve-devel] [RFC PATCH pve-installer 6/6] closes #5757: common: add checks for valid IPv4 address within subnet

2025-04-28 Thread Michael Köppl
On 4/28/25 12:22, Christoph Heiss wrote: At least for IPv4 /31 prefixes, this check (including the network address part) is wrong. RFC 3021 [0] explicitly allows such subnets for point-to-point links. E.g. 192.168.0.0/31 is a valid subnet with 2 hosts, 192.168.0.0/31 and 192.168.0.1/31. [0] htt

Re: [pve-devel] [PATCH ha-manager 09/15] manager: apply colocation rules when selecting service nodes

2025-04-28 Thread Fiona Ebner
Am 28.04.25 um 14:26 schrieb Fiona Ebner: > Am 25.03.25 um 16:12 schrieb Daniel Kral: >> + >> +delete $allowed_nodes->{$node}; >> +} >> +} elsif (scalar keys %$possible_nodes) { >> +# limit to the possible nodes the service should be on, if there are >> any. >> +for my $nod

Re: [pve-devel] [PATCH ha-manager 13/15] test: ha tester: add test cases for loose colocation rules

2025-04-28 Thread Fiona Ebner
Am 25.03.25 um 16:12 schrieb Daniel Kral: > Add test cases for loose positive and negative colocation rules, i.e. > where services should be kept on the same node together or kept separate > nodes. These are copies of their strict counterpart tests, but verify > the behavior if the colocation rule

Re: [pve-devel] [PATCH pve-installer 2/6] common: use get_min_disks as single source of truth for RAID config checks

2025-04-28 Thread Michael Köppl
On 4/28/25 13:48, Christoph Heiss wrote: proxmox-installer-common/src/disk_checks.rs | 18 -- 1 file changed, 4 insertions(+), 14 deletions(-) diff --git a/proxmox-installer-common/src/disk_checks.rs b/proxmox-installer-common/src/disk_checks.rs index ecc43bd..1d17e2c 100644

[pve-devel] [PATCH pve_flutter_frontend] ui: replace deprecated `withOpacity` function

2025-04-28 Thread Shan Shaji
As of flutter v3.27 the `withOpacity` function is deprecated. This patch replaces all occurrences of `withOpacity` with `withValues` function. Signed-off-by: Shan Shaji --- lib/main.dart | 4 ++-- lib/pages/main_layout_slim.dart| 12 +