[pve-devel] [PATCH docs] faq: change to inline anchors to land on the actual questions

Signed-off-by: Alexander Zeidler --- pve-faq.adoc | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/pve-faq.adoc b/pve-faq.adoc index 93fef32..2388673 100644 --- a/pve-faq.adoc +++ b/pve-faq.adoc @@ -77,8 +77,7 @@ CPU. It is not limited to Linux guests but allows

[pve-devel] [PATCH docs] ha-manager, software updates: add useful BlockIds

Signed-off-by: Alexander Zeidler --- ha-manager.adoc | 5 + system-software-updates.adoc | 1 + 2 files changed, 6 insertions(+) diff --git a/ha-manager.adoc b/ha-manager.adoc index 8390782..53330a1 100644 --- a/ha-manager.adoc +++ b/ha-manager.adoc @@ -260,6 +260,7 @@ This all

[pve-devel] [PATCH docs] sysadmin: revise firmware chapter, add firmware repo section

interpret versions and how to recover a possibly unbootable system * move info about non-free-firmware repo to "Package Repositories" Chapter "Package Repositories": * add new section "Debian Firmware Repository" Signed-off-by: Alexander Zeidler --- Depends on "[

[pve-devel] [PATCH i18n] update German translation

Signed-off-by: Alexander Zeidler --- de.po | 183 ++ 1 file changed, 81 insertions(+), 102 deletions(-) diff --git a/de.po b/de.po index 5bc5e42..ceadce6 100644 --- a/de.po +++ b/de.po @@ -8,7 +8,7 @@ msgstr "" "Pro

Re: [pve-devel] [PATCH i18n] update German translation

On Wed, 2023-11-22 at 16:27 +0100, Alexander Zeidler wrote: > Signed-off-by: Alexander Zeidler > --- >  de.po | 183 Hannes Duerr mentioned: 1. that my sent translation "Severität" for Severity/Severities is maybe not as appropriate as "Schweregrad". "Schwer

[pve-devel] [PATCH i18n 3/3] update German translation

Signed-off-by: Alexander Zeidler --- de.po | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/de.po b/de.po index 920ae78..3217a20 100644 --- a/de.po +++ b/de.po @@ -643,7 +643,7 @@ msgstr "Möchten Sie den Snapshot {0} wirklich entfernen?" #: proxmox-backu

[pve-devel] [PATCH i18n 2/3] german: add missing translations

Signed-off-by: Alexander Zeidler --- de.po | 33 ++--- 1 file changed, 18 insertions(+), 15 deletions(-) diff --git a/de.po b/de.po index 38f81e5..920ae78 100644 --- a/de.po +++ b/de.po @@ -24,11 +24,11 @@ msgstr "(Kein Startgerät ausgewählt)" #: prox

[pve-devel] [PATCH i18n 1/3] german: unify different words of the same meaning

Signed-off-by: Alexander Zeidler --- de.po | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/de.po b/de.po index ccc2306..38f81e5 100644 --- a/de.po +++ b/de.po @@ -1439,13 +1439,13 @@ msgstr "Wechsler" #: pve-manager/www/m

[pve-devel] [PATCH docs 2/2] installation: be more specific about current secure boot state

Signed-off-by: Alexander Zeidler --- pve-installation.adoc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pve-installation.adoc b/pve-installation.adoc index 44b59c0..1e909e2 100644 --- a/pve-installation.adoc +++ b/pve-installation.adoc @@ -60,7 +60,8 @@ Please insert the

[pve-devel] [PATCH docs 1/2] secure boot: fix typos, add inline code formatting

Signed-off-by: Alexander Zeidler --- system-booting.adoc | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/system-booting.adoc b/system-booting.adoc index c8761a4..af56225 100644 --- a/system-booting.adoc +++ b/system-booting.adoc @@ -380,10 +380,11 @@ and

[pve-devel] [PATCH manager] pve7to8: fix and improve wording

Signed-off-by: Alexander Zeidler --- bin/Makefile | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/bin/Makefile b/bin/Makefile index 06d8148e..2185b009 100644 --- a/bin/Makefile +++ b/bin/Makefile @@ -62,10 +62,11 @@ pve6to7.1: pve7to8.1: printf &quo

Re: [pve-devel] [PATCH proxmox-i18n] use xgettext to extract translatable strings

>From a brief look at it: I also think it's a good idea to provide more information for translators (where it actually adds value and doesn't just bloat code). > Cons: > - In total 3 translations were marked as fuzzy. Translators will have to > review and mark them as translated again. since ~1

[pve-devel] [PATCH manager 1/3] pvesubscription: add missing return statement

Signed-off-by: Alexander Zeidler --- PVE/CLI/pvesubscription.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/PVE/CLI/pvesubscription.pm b/PVE/CLI/pvesubscription.pm index 2eb26cb4..2fd641d0 100755 --- a/PVE/CLI/pvesubscription.pm +++ b/PVE/CLI/pvesubscription.pm @@ -47,6 +47,8

[pve-devel] [PATCH manager 3/3] report: format iptables output for readability

Signed-off-by: Alexander Zeidler --- PVE/Report.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/Report.pm b/PVE/Report.pm index 10b28c79..9c46aa9b 100644 --- a/PVE/Report.pm +++ b/PVE/Report.pm @@ -85,7 +85,7 @@ my $init_report_cmds = sub { cmds

[pve-devel] [PATCH manager 2/3] pvesubscription: update set-offline-key description

and point users to proxmox-offline-mirror-helper Signed-off-by: Alexander Zeidler --- PVE/CLI/pvesubscription.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/PVE/CLI/pvesubscription.pm b/PVE/CLI/pvesubscription.pm index 2fd641d0..f1c11289 100755 --- a/PVE/CLI

Re: [pve-devel] [PATCH manager 1/3] pvesubscription: add missing return statement

On Wed, 2024-01-10 at 10:29 +0100, Thomas Lamprecht wrote: > any reason this is relevant you might want to add to the commit > message here? to avoid a failing null check and its error message. This confused users since the activation was successful anyway. > simple > > return; > > is slightly

Re: [pve-devel] [PATCH manager 2/3] pvesubscription: update set-offline-key description

On Wed, 2024-01-10 at 10:41 +0100, Thomas Lamprecht wrote: > Am 09/01/2024 um 15:23 schrieb Alexander Zeidler: > > and point users to proxmox-offline-mirror-helper > > the change is fine, but having any background/reason/why here could be > still nice, Thanks! Since a cu

[pve-devel] [PATCH docs] pvecm: correct network address

Signed-off-by: Alexander Zeidler --- pvecm.adoc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pvecm.adoc b/pvecm.adoc index 14556c2..48d86bc 100644 --- a/pvecm.adoc +++ b/pvecm.adoc @@ -661,7 +661,7 @@ hostnames, ensure that they are resolvable from all nodes (see also

[pve-devel] [PATCH docs] ballooning example: improve wording

Signed-off-by: Alexander Zeidler --- qm.adoc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/qm.adoc b/qm.adoc index fa6a772..c2392f3 100644 --- a/qm.adoc +++ b/qm.adoc @@ -689,8 +689,7 @@ database VM when spare RAM is available. For this you assign a Shares property of

[pve-devel] [PATCH docs 1/3] qm: resource limits: revise section cpulimit

* precise statements * increase compactness w/o complexity * improve section-formatting Signed-off-by: Alexander Zeidler --- qm.adoc | 40 +--- 1 file changed, 21 insertions(+), 19 deletions(-) diff --git a/qm.adoc b/qm.adoc index c2392f3..268a635 100644

[pve-devel] [PATCH docs 2/3] qm: resource limits: revise section cpuunits

* restructure sentences minimally * improve section-formatting Signed-off-by: Alexander Zeidler --- qm.adoc | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/qm.adoc b/qm.adoc index 268a635..15aa0fc 100644 --- a/qm.adoc +++ b/qm.adoc @@ -364,17 +364,21

[pve-devel] [PATCH docs 3/3] qm: resource limits: revise section affinity

* mention side effects * increase compactness w/o complexity * improve section-formatting Signed-off-by: Alexander Zeidler --- qm.adoc | 32 ++-- 1 file changed, 18 insertions(+), 14 deletions(-) diff --git a/qm.adoc b/qm.adoc index 15aa0fc..d92db75 100644 --- a

Re: [pve-devel] [PATCH docs] ballooning example: improve wording

On Tue, 2024-01-16 at 14:28 +0100, Hannes Dürr wrote: > I don't think this is an improvement of the wording, liked the previous > version more. > Are there people who have not understood the wording? It's not an extra (additional) amount of memory for this VM, but its total. Therefore, using the w

[pve-devel] [PATCH docs] ballooning: precise example

Signed-off-by: Alexander Zeidler --- qm.adoc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/qm.adoc b/qm.adoc index 4ff5014..86762d3 100644 --- a/qm.adoc +++ b/qm.adoc @@ -698,9 +698,9 @@ database blocks in the database server RAM, you would like to prioritize the

Re: [pve-devel] [PATCH docs] ballooning example: improve wording

On Tue, 2024-01-16 at 14:36 +0100, Alexander Zeidler wrote: > On Tue, 2024-01-16 at 14:28 +0100, Hannes Dürr wrote: > > I don't think this is an improvement of the wording, liked the previous > > version more. > > Are there people who have not understood the wo

[pve-devel] [PATCH v2 manager] pvesubscription: add missing return statement

to avoid a failing null check and its error message. This confused users since the activation was successful anyway. Signed-off-by: Alexander Zeidler --- v2: * use "return" instead of "return undef" * add commit message v1: https://lists.proxmox.com/pipermail/pve-devel/2024

Re: [pve-devel] [PATCH manager 1/3] pvesubscription: add missing return statement

On Wed, 2024-01-10 at 11:59 +0100, Alexander Zeidler wrote: > On Wed, 2024-01-10 at 10:29 +0100, Thomas Lamprecht wrote: > > any reason this is relevant you might want to add to the commit > > message here? > to avoid a failing null check and its error message. This confused

[pve-devel] [PATCH manager 4/9] report: add `apt-cache policy` to list recognized APT sources

Packages release o=Proxmox,a=stable,n=bookworm,l=Proxmox VE Enterprise Debian Repository,c=pve-enterprise,b=amd64 ... Pinned packages: intel-microcode -> 3.20231114.1~deb12u1 with priority 1234 Signed-off-by: Alexander Zeidler --- PVE/Report.pm | 1 + 1 file changed, 1 insertion(+

[pve-devel] [PATCH manager 1/9] report: add kernel command line including boot time

While using `/proc/cmdline` would already provide an initial info for debugging passthrough and similar, the use of `dmesg` is an easy way to get the boot date as an absolute value for free (additional to the relative value in `uptime` from `top`). Signed-off-by: Alexander Zeidler --- PVE

[pve-devel] [PATCH manager 5/9] report: move `lscpu` & cluster info to more appropriate sections

Signed-off-by: Alexander Zeidler --- PVE/Report.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PVE/Report.pm b/PVE/Report.pm index 2c2a5e12..505629c7 100644 --- a/PVE/Report.pm +++ b/PVE/Report.pm @@ -41,8 +41,6 @@ my $init_report_cmds = sub { 

[pve-devel] [PATCH manager 3/9] report: add list of upgradable packages

8.1.6 amd64 [upgradable from: 8.1.4] Signed-off-by: Alexander Zeidler --- PVE/Report.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/PVE/Report.pm b/PVE/Report.pm index 6014f13e..ad5c2aa0 100644 --- a/PVE/Report.pm +++ b/PVE/Report.pm @@ -36,6 +36,7 @@ my $init_report_cmds = sub

[pve-devel] [PATCH manager 2/9] report: add `jobs.cfg` to debug related network/load/backup/etc issues

Suggested-by: Friedrich Weber Signed-off-by: Alexander Zeidler --- PVE/Report.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/PVE/Report.pm b/PVE/Report.pm index d4191769..6014f13e 100644 --- a/PVE/Report.pm +++ b/PVE/Report.pm @@ -35,6 +35,7 @@ my $init_report_cmds = sub

[pve-devel] [PATCH manager 6/9] report: switch `dmidecode` to quiet to omit almost never needed info

like on this system: # dmidecode -t bios # dmidecode 3.4 Getting SMBIOS data from sysfs. SMBIOS 3.0.0 present. Handle 0x, DMI type 0, 24 bytes Handle 0x005C, DMI type 13, 22 bytes Signed-off-by: Alexander Zeidler --- PVE/Report.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[pve-devel] [PATCH manager 8/9] report: add info about (un)used memory slots

Installed Signed-off-by: Alexander Zeidler --- PVE/Report.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/PVE/Report.pm b/PVE/Report.pm index f28d7b38..fe497b43 100644 --- a/PVE/Report.pm +++ b/PVE/Report.pm @@ -106,6 +106,7 @@ my $init_report_cmds = sub { cmds

[pve-devel] [PATCH manager 9/9] report: add microcode info to better assess possible system impacts

* list availability and installation status of `*microcode` packages * grep for applied "Early OS Microcode Updates" * grep for (un)patched CPU vulnerability messages Signed-off-by: Alexander Zeidler --- PVE/Report.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/PVE/Repor

[pve-devel] [PATCH manager 7/9] report: add info of the mainboard in use

Signed-off-by: Alexander Zeidler --- PVE/Report.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/PVE/Report.pm b/PVE/Report.pm index 3a81bdb2..f28d7b38 100644 --- a/PVE/Report.pm +++ b/PVE/Report.pm @@ -104,6 +104,7 @@ my $init_report_cmds = sub { hardware => { or

Re: [pve-devel] [PATCH manager 8/9] report: add info about (un)used memory slots

On Fri, 2024-03-22 at 14:59 +0100, Alexander Zeidler wrote: > * to see if a RAM upgrade is slot/capacity-wise possible > * to spot added/replaced RAM that may now be causing issues > > # dmidecode -t memory ... > Maximum Capacity: 2 TB > Number Of Devices: 8 &

Re: [pve-devel] [PATCH manager 9/9] report: add microcode info to better assess possible system impacts

On Fri, 2024-03-22 at 17:44 +0100, Stoiko Ivanov wrote: > On Fri, 22 Mar 2024 14:59:33 +0100 > Alexander Zeidler wrote: > > > * list availability and installation status of `*microcode` packages > > * grep for applied "Early OS Microcode Updates" > > * g

Re: [pve-devel] [PATCH manager 1/9] report: add kernel command line including boot time

On Mon, 2024-03-25 at 08:52 +0100, Thomas Lamprecht wrote: > > > > On 22/03/2024 14:59, Alexander Zeidler wrote: > > > > > > > > While using `/proc/cmdline` would already provide an initial > > > > > > > > info for > > >

Re: [pve-devel] [PATCH manager 2/9] report: add `jobs.cfg` to debug related network/load/backup/etc issues

On Mon, 2024-03-25 at 09:08 +0100, Thomas Lamprecht wrote: > > > > On 22/03/2024 14:59, Alexander Zeidler wrote: > > > > > > > > Suggested-by: Friedrich Weber > > > > > > > > Signed-off-by: Alexander Zeidler > > > > > >

Re: [pve-devel] [PATCH manager 3/9] report: add list of upgradable packages

On Mon, 2024-03-25 at 09:02 +0100, Thomas Lamprecht wrote: > On 22/03/2024 14:59, Alexander Zeidler wrote: > > * to easily see if APT already knows about old packages in use and > > their exact version > > * to reconsider asking for applying updates as a first recommendation

Re: [pve-devel] [PATCH manager 5/9] report: move `lscpu` & cluster info to more appropriate sections

be good.. > > On 22/03/2024 14:59, Alexander Zeidler wrote: > > Signed-off-by: Alexander Zeidler > > --- > > PVE/Report.pm | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/PVE/Report.pm b/PVE/Report.pm > > index 2

Re: [pve-devel] [PATCH manager 6/9] report: switch `dmidecode` to quiet to omit almost never needed info

On Mon, 2024-03-25 at 09:20 +0100, Thomas Lamprecht wrote: > On 22/03/2024 14:59, Alexander Zeidler wrote: > > like on this system: > > > > # dmidecode -t bios > > # dmidecode 3.4 > > Getting SMBIOS data from sysfs. > > SMBIOS 3.0.0 present. > &

Re: [pve-devel] [PATCH manager 7/9] report: add info of the mainboard in use

OEM Information: 0x Height: 2 U -> Number Of Power Cords: 2 Contained Elements: 0 > > On 2024-03-22 14:59, Alexander Zeidler wrote: > > Signed-off-by: Alexander Zeidler > > --- > > PVE/Report.pm | 1 + > > 1 file changed, 1 insert

Re: [pve-devel] [PATCH manager 9/9] report: add microcode info to better assess possible system impacts

On Mon, 2024-03-25 at 10:00 +0100, Thomas Lamprecht wrote: > On 22/03/2024 14:59, Alexander Zeidler wrote: > > * list availability and installation status of `*microcode` packages > > * grep for applied "Early OS Microcode Updates" > > * grep for (un)pat

[pve-devel] [PATCH manager] api: apt versions: track optional amd64/intel-microcode packages

Signed-off-by: Alexander Zeidler --- PVE/API2/APT.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/PVE/API2/APT.pm b/PVE/API2/APT.pm index 54121ec2..4ab6da60 100644 --- a/PVE/API2/APT.pm +++ b/PVE/API2/APT.pm @@ -759,6 +759,7 @@ __PACKAGE__->register_method({ push @list, s

[pve-devel] [PATCH manager] pve7to8: reword and fix typos in description

Signed-off-by: Alexander Zeidler --- bin/Makefile | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/bin/Makefile b/bin/Makefile index 180a91b5..aa149c06 100644 --- a/bin/Makefile +++ b/bin/Makefile @@ -66,10 +66,10 @@ pve6to7.1: pve7to8.1: printf ".TH PV

[pve-devel] [PATCH manager] pveversion: show upgradable package version in verbose output

urrent System Report includes pveversion -v # pveversion -v proxmox-ve: 8.1.0 (running kernel: 6.5.13-5-pve) pve-manager: 8.1.6 (running version: 8.1.6/b7e8e914a1db70cc) [available: 8.1.10] ... vncterm: 1.8.0 zfsutils-linux: 2.2.3-pve1 [available: 2.2.3-pve2] Signed-off-by: Alexander Zeidler ---

[pve-devel] [PATCH manager] report: fix regex of config filenames

to only match those that are correct/accepted by their software Signed-off-by: Alexander Zeidler --- PVE/Report.pm | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/PVE/Report.pm b/PVE/Report.pm index 9c46aa9b..53ffdcbb 100644 --- a/PVE/Report.pm +++ b/PVE/Report.pm

[pve-devel] [PATCH manager 2/7] report: create "jobs" section, add `jobs.cfg`

to recognize temporal correlations with network/load/backup/etc issues Suggested-by: Friedrich Weber Signed-off-by: Alexander Zeidler --- v2: * move away from "general" section v1: https://lists.proxmox.com/pipermail/pve-devel/2024-March/062346.html PVE/Report.pm | 6 ++ 1 fi

[pve-devel] [PATCH manager 6/7] report: add info about (un)used memory slots

: 18ASF2G72PZ-2G6D1 Size: 16 GB Part Number: 18ASF2G72PZ-2G6D1 Size: No Module Installed Size: No Module Installed Size: No Module Installed Size: No Module Installed Signed-off-by: Alexander Zeidler --- v2: * make regex stricter * reduce possible dmidecode

[pve-devel] [PATCH manager 1/7] report: add kernel command line from current boot

to get a first clue for debugging passthrough and similar issues, when no dmesg output has been provided yet. Signed-off-by: Alexander Zeidler --- v2: * move away from dmesg base * only print kernel command line (boot times can be added by another patch) v1: https://lists.proxmox.com/pipermail

[pve-devel] [PATCH manager 4/7] report: list held back packages

Signed-off-by: Alexander Zeidler --- v2: * newly added PVE/Report.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/PVE/Report.pm b/PVE/Report.pm index 9b6cd95c..1ed91c8e 100644 --- a/PVE/Report.pm +++ b/PVE/Report.pm @@ -39,6 +39,7 @@ my $init_report_cmds = sub { sub

[pve-devel] [PATCH manager 3/7] report: add `apt-cache policy` to list recognized APT sources

Packages release o=Proxmox,a=stable,n=bookworm,l=Proxmox VE Enterprise Debian Repository,c=pve-enterprise,b=amd64 ... Pinned packages: intel-microcode -> 3.20231114.1~deb12u1 with priority 1234 Signed-off-by: Alexander Zeidler --- Expects applied: report: fix regex of config fi

[pve-devel] [PATCH manager 5/7] report: overhaul `dmidecode` related output

Power Supply State: Critical Thermal State: Safe Number Of Power Cords: 2 (...) and `-t 32`: System Boot Information Status: Firmware-detected hardware failure which can hint to Proxmox-independant issues, debug-able via IPMI. Signed-off-by: Alexander Zeidler --

[pve-devel] [PATCH manager 7/7] report: add recent boot timestamps which may show fencing/crash events

u Apr 11 16:31:12 2024 (00:05) ... Furthermore, it shows the booted/crashed/problematic kernel version. `last` is also used since currently `journalctl --list-boots` can take 10 seconds or even longer on some systems, with no option to limit the amount of reported boot lines. Signed-off-by: Alexander Zeidler

Re: [pve-devel] [PATCH manager 1/7] report: add kernel command line from current boot

Sorry, this is of course v2. ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH manager 5/7] report: overhaul `dmidecode` related output

On Thu, 2024-04-18 at 12:24 +0200, Mira Limbeck wrote: > Would it be possible to align these correctly, or just use a single > space between type and value? > I'd prefer explicit commands like: Better formatting can of course be achieved, see the adapted example below, followed by your mentioned

Re: [pve-devel] [PATCH manager 6/7] report: add info about (un)used memory slots

Size: No Module Installed > It might be better to keep that command in a cheatsheet to ask for it if > actually needed instead of polluting the report :) > > On 2024-04-18 11:16, Alexander Zeidler wrote: > > * to see if a RAM upgrade is slot/capacity-wise possible > >

Re: [pve-devel] [PATCH manager 6/7] report: add info about (un)used memory slots

On Thu, 2024-04-18 at 12:20 +0200, Mira Limbeck wrote: > On 4/18/24 11:16, Alexander Zeidler wrote: > > * to see if a RAM upgrade is slot/capacity-wise possible > > * to spot added/replaced RAM that may now be causing issues > > > > Maximum Capacity: 2 TB > >

Re: [pve-devel] [PATCH manager 7/7] report: add recent boot timestamps which may show fencing/crash events

On Thu, 2024-04-18 at 12:43 +0200, Mira Limbeck wrote: > On 4/18/24 11:16, Alexander Zeidler wrote: > > Successful boots which crashed somehow and sometime afterwards, will > > show the same "until" value ("still running" or timestamp) as the next > > fol

Re: [pve-devel] [PATCH manager 1/7] report: add kernel command line from current boot

On Thu, 2024-04-18 at 13:05 +0200, Mira Limbeck wrote: > On 4/18/24 11:16, Alexander Zeidler wrote: > > to get a first clue for debugging passthrough and similar issues, when > > no dmesg output has been provided yet. > > > > Signed-off-by: Alexander Zeidler > >

Re: [pve-devel] [PATCH manager 1/9] report: add kernel command line including boot time

v2 of this series: [PATCH manager 1/7] report: add kernel command line from current boot https://lists.proxmox.com/pipermail/pve-devel/2024-April/063282.html ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailm

[pve-devel] [PATCH v3 manager 1/2] report: overhaul `dmidecode` related output

ted hardware failure which can hint to Proxmox-independant issues, debug-able via IPMI. Signed-off-by: Alexander Zeidler --- v3: * as discussed also off-list, switch to `cat` for board and product output to avoid complexity v2: https://lists.proxmox.com/pipermail/pve-devel/2024-April/063284

[pve-devel] [PATCH v3 manager 2/2] report: add recent boot timestamps which may show fencing/crash events

u Apr 11 16:31:12 2024 (00:05) ... Furthermore, it shows the booted/crashed/problematic kernel version. `last` is also used since currently `journalctl --list-boots` can take 10 seconds or even longer on some systems, with no option to limit the amount of reported boot lines. Signed-off-by: Alexander Zeidler

[pve-devel] [PATCH v2 manager] pve7to8: reword and fix typos in description

Signed-off-by: Alexander Zeidler --- v2: * incorporate all feedback from v1 v1: https://lists.proxmox.com/pipermail/pve-devel/2024-April/063252.html bin/Makefile | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/bin/Makefile b/bin/Makefile index 180a91b5..6c5f9b14

[pve-devel] [PATCH manager] pveversion: fix whitespaces

Signed-off-by: Alexander Zeidler --- bin/pveversion | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bin/pveversion b/bin/pveversion index a98d2c66..d32a6a0c 100755 --- a/bin/pveversion +++ b/bin/pveversion @@ -60,7 +60,7 @@ my $opt_verbose; if (!GetOptions ('ve

[pve-devel] [PATCH manager 2/4] add 'pbs-entries-max' to permission check & config

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- PVE/API2/Backup.pm | 2 +- configs/vzdump.conf | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/PVE

[pve-devel] [PATCH container 3/4] add 'pbs-entries-max' parameter

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- src/PVE/VZDump/LXC.pm | 5 + 1 file changed, 5 insertions(+) diff --git a/src/PVE/VZDump/LXC.pm b/src/PVE/VZDump

[pve-devel] [PATCH guest-common 4/4] vzdump: schema: add 'pbs-entries-max' property

Signed-off-by: Alexander Zeidler --- src/PVE/VZDump/Common.pm | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/PVE/VZDump/Common.pm b/src/PVE/VZDump/Common.pm index a6fe483..28ab0d3 100644 --- a/src/PVE/VZDump/Common.pm +++ b/src/PVE/VZDump/Common.pm @@ -282,6 +282,12 @@ my

[pve-devel] [PATCH manager/container/guest-common 0/4] fix #3069: add pbs-entries-max to vzdump.conf

; permissions. This patch series makes the new optional parameter 'pbs-entries-max' available in vzdump and vzdump.conf. [1] https://bugzilla.proxmox.com/show_bug.cgi?id=3069 pve-manager: Alexander Zeidler (2): api: backup: refactor backup permission check add 'pbs-entrie

[pve-devel] [PATCH manager 1/4] api: backup: refactor backup permission check

Unify style before adding another parameter check Signed-off-by: Alexander Zeidler --- PVE/API2/Backup.pm | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/PVE/API2/Backup.pm b/PVE/API2/Backup.pm index 45eb47e2..cae889f4 100644 --- a/PVE/API2/Backup.pm +++ b/PVE/API2

[pve-devel] [PATCH v2 manager 2/4] api: backup: add 'pbs-entries-max' to permission check & config

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- Changes from v1: Add 'pbs-entries-max' according to the new shortened permission check PVE/API2/Back

[pve-devel] [PATCH v2 manager 1/4] api: backup: refactor backup permission check

Alter style to make the parameter check more concise Signed-off-by: Alexander Zeidler --- Changes from v1: Shorten permission check PVE/API2/Backup.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/API2/Backup.pm b/PVE/API2/Backup.pm index 45eb47e2..70753c2e 100644

[pve-devel] [PATCH v2 guest-common 4/4] vzdump: schema: add 'pbs-entries-max' property

Signed-off-by: Alexander Zeidler --- Changes from v1: Improve description Move description to 'performance' section Remove arrow alignment src/PVE/VZDump/Common.pm | 10 ++ 1 file changed, 10 insertions(+) diff --git a/src/PVE/VZDump/Common.pm b/src/PVE/VZDump/Common.pm ind

[pve-devel] [PATCH v2 container 3/4]: add 'pbs-entries-max' parameter

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- Changes from v1: Reword loginfo message and include new set value Fix indentation src/PVE/VZDump/LXC.pm | 5 + 1

[pve-devel] [PATCH v2 manager/container/guest-common 0/4] fix #3069: add pbs-entries-max to vzdump.conf

; permissions. This patch series makes the new optional parameter 'pbs-entries-max' available in vzdump and vzdump.conf. [1] https://bugzilla.proxmox.com/show_bug.cgi?id=3069 Changes from v1: Incorporated Fiona's comments. For details see patch emails. pve-manager: Alexander Z

[pve-devel] [PATCH docs] sysadmin: add section 'Firmware Updates' and references

Firmware updates are important, their existence should not be checked only when there are already noticeable problems. Signed-off-by: Alexander Zeidler --- firmware-updates.adoc | 60 +++ local-lvm.adoc| 4 ++- qm.adoc | 15

Re: [pve-devel] [PATCH docs] sysadmin: add section 'Firmware Updates' and references

On Wed, 2023-07-26 at 12:58 +0200, Aaron Lauterer wrote: > Maybe something like this? Great! Will also fix my overlooked line lengths. ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH v3 manager/container/guest-common/docs 1/5] api: backup: refactor backup permission check

Alter style to make the parameter check more concise Signed-off-by: Alexander Zeidler --- Changes since v2: * none Changes since v1: Apply Fiona's comments: * shorten permission check PVE/API2/Backup.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE

[pve-devel] [PATCH v3 guest-common 4/5] fix #3069: vzdump: schema: add 'pbs-entries-max' property

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- Changes since v2: Apply Fiona's comments: * rephrase description * fix dot style nit Changes since v1: * im

[pve-devel] [PATCH v3 docs 5/5] vzdump: mention new parameter 'pbs-entries-max'

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- Changes since v2: * add this commit vzdump.adoc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a

[pve-devel] [PATCH v3 manager 2/5] fix #3069: vzdump: add property 'performance: pbs-entries-max=N'

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- Changes since v2: Apply Fiona's comments: * remove property from permission check since it's included in p

[pve-devel] [PATCH v3 container 3/5]: fix #3069: vzdump: add 'pbs-entries-max' parameter

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- Changes since v2: Apply Fiona's comments: * fix parameter check * fix both style nits Changes since v1: * r

[pve-devel] [PATCH v3 manager/container/guest-common/docs 0/5] fix #3069: add pbs-entries-max to vzdump.conf

fy' permissions. This patch series makes the new optional performance parameter 'pbs-entries-max' available in vzdump and vzdump.conf. [1] https://bugzilla.proxmox.com/show_bug.cgi?id=3069 Signed-off-by: Alexander Zeidler --- Changes since v2: * apply Fiona's comments. Fo

[pve-devel] [PATCH v2 docs] sysadmin: add section 'Firmware Updates' and references

Firmware updates are important, their existence should not be checked only when there are already noticeable problems. Signed-off-by: Alexander Zeidler --- Changes since v1: * apply Aaron's comment (section Vendor-specific) firmware-updates.adoc

[pve-devel] [PATCH v4 manager 1/4] fix #3069: vzdump: add property 'performance: pbs-entries-max=N'

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- Changes since v3: Apply Fiona's comment: * use only one performance line Changes since v2: Apply Fiona'

[pve-devel] [PATCH v4 manager/container/guest-common/docs 0/4] fix #3069: add pbs-entries-max to vzdump.conf

fy' permissions. This patch series makes the new optional performance parameter 'pbs-entries-max' available in vzdump and vzdump.conf. [1] https://bugzilla.proxmox.com/show_bug.cgi?id=3069 Signed-off-by: Alexander Zeidler --- Changes since v3: * apply Fiona's comments. For

[pve-devel] [PATCH v4 docs 4/4] vzdump: mention new parameter 'pbs-entries-max'

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- Changes since v3: * none Changes since v2: * add this commit vzdump.adoc | 3 ++- 1 file changed, 2 insertions

[pve-devel] [PATCH v4 guest-common 3/4] fix #3069: vzdump: schema: add 'pbs-entries-max' property

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- Changes since v3: Apply Fiona's comments: * simplify description Changes since v2: Apply Fiona'

[pve-devel] [PATCH v4 container 2/4]: fix #3069: vzdump: add 'pbs-entries-max' parameter

configuring pbs-entries-max can avoid failing backups due to a high amount of files in folders where a folder exclusion is not possible Signed-off-by: Alexander Zeidler --- Changes since v3: Apply Fiona's comment: * fix style (https://lists.proxmox.com/pipermail/pve-devel/2023-Ja

Re: [pve-devel] [PATCH v2 docs] sysadmin: add section 'Firmware Updates' and references

On Fri, 2023-07-28 at 15:21 +0200, Alexander Zeidler wrote: > Firmware updates are important, their existence should not be checked > only when there are already noticeable problems. > > Signed-off-by: Alexander Zeidler Ping. Still applies on master. Would be of interest, see e.g.

Re: [pve-devel] [PATCH v4 manager/container/guest-common/docs 0/4] fix #3069: add pbs-entries-max to vzdump.conf

On Fri, 2023-07-28 at 15:28 +0200, Alexander Zeidler wrote: > Users reported[1] about failing container backups to PBS when having > a large amount of files in a directory. > > proxmox-backup-client has already 'entries-max' implemented. The > default value prevents out o

[pve-devel] [PATCH manager] ui: vm create wizard: default to 2 cores for win 11 os type

-86c11283-ea52-4782-9efd-7674389a7ba3 [1] https://www.microsoft.com/software-download/windows11 [2] https://learn.microsoft.com/en-us/windows-server/get-started/hardware-requirements [3] https://www.microsoft.com/en-us/evalcenter/download-windows-server-2022 Signed-off-by: Alexander Zeidler --- www

[pve-devel] [PATCH installer] assistant: prepare iso: s/direct/included to match current naming

Signed-off-by: Alexander Zeidler --- proxmox-auto-install-assistant/src/main.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/proxmox-auto-install-assistant/src/main.rs b/proxmox-auto-install-assistant/src/main.rs index 17f8e96..c14316c 100644 --- a/proxmox-auto

[pve-devel] [PATCH installer] assistant: fix spelling and precise text in help usage output

Signed-off-by: Alexander Zeidler --- proxmox-auto-install-assistant/src/main.rs | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/proxmox-auto-install-assistant/src/main.rs b/proxmox-auto-install-assistant/src/main.rs index 9bc083e..0debd29 100644 --- a/proxmox-auto

[pve-devel] [PATCH installer] install module: getters: correctly use plural in error messages

Signed-off-by: Alexander Zeidler --- Proxmox/Install.pm | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Proxmox/Install.pm b/Proxmox/Install.pm index 6ad9d17..c0f8955 100644 --- a/Proxmox/Install.pm +++ b/Proxmox/Install.pm @@ -255,7 +255,7 @@ sub get_zfs_raid_setup

Re: [pve-devel] [PATCH docs v2] pveceph: document cluster shutdown

On Wed, 2024-05-22 at 10:33 +0200, Aaron Lauterer wrote: > Signed-off-by: Aaron Lauterer > --- > changes since v1: > * incorporated suggested changes in phrasing to fix grammar and > distinguish the steps on how to power down the nodes better > > pveceph.adoc | 50 +

Re: [pve-devel] [PATCH docs v3] pveceph: document cluster shutdown

On Tue, 2024-05-28 at 13:54 +0200, Aaron Lauterer wrote: > Signed-off-by: Aaron Lauterer > --- > changes: > incorporated additional feedback regarding phrasing, structure and > spelling Reviewed-by: Alexander Zeidler ___ pve-devel m

Re: [pve-devel] [PATCH kernel 2/2] cherry-pick potential fix for NULL pointer deref with AMD Arcturus GPU during boot

> On 10.07.2024 11:37 GMT Fiona Ebner wrote: > The issue was reported in the enterprise support and is handled by > Alexander Zeidler. It has the following trace [0] and causes an issue > with the networking down the line, because 'udevadm settle' would time > out. T

  1   2   >