[pve-devel] Superseded: Re: [PATCH installer v2 0/3] remove mirror selection from installer

2025-09-22 Thread Shannon Sterz
On Mon Sep 22, 2025 at 11:08 AM CEST, Shannon Sterz wrote: > these three patches remove the debian mirror selection from the > installer as it is no longer needed. debian recommends using its CDN > these days [1]. so we don't need to select a mirror. > > [1]: > https://www.debian.org/releases/trix

[pve-devel] applied: [PATCH common 1/4] pty: introduce read_line() function

2025-09-22 Thread Thomas Lamprecht
On Thu, 18 Sep 2025 15:50:53 +0200, Fiona Ebner wrote: > [FE: avoid changes to read_password() > rename from read_input() to read_line() > avoid undef warning if there is no input by avoiding chomp then > print prompt even if not reading from interactive for better log] > > Applie

Re: [pve-devel] [PATCH pve-manager 3/5] api: add APT versions return schema

2025-09-22 Thread Thomas Lamprecht
Am 19.09.25 um 11:40 schrieb n.f...@proxmox.com: > From: Nicolas Frey > > Signed-off-by: Nicolas Frey > --- > PVE/API2/APT.pm | 61 - > 1 file changed, 60 insertions(+), 1 deletion(-) > > diff --git a/PVE/API2/APT.pm b/PVE/API2/APT.pm > index 0d0

[pve-devel] [PATCH installer 1/1] fix #2164: add option to define a vlan tag within the installer

2025-09-22 Thread Florian Paul Azim Hoberg via pve-devel
--- Begin Message --- To ensure a Promox node can access a desired network resource which requires a given vlan tag, this feature adds the possibility to optionally define a vlan tag within the auto installer, tui and gui. Signed-off-by: Florian Paul Azim Hoberg @gyptazy --- initial commit: *

[pve-devel] applied: [PATCH qemu-server] fix #6713: snapshot volume chain: fix snapshot after disk move with zeroinit

2025-09-22 Thread Thomas Lamprecht
On Fri, 19 Sep 2025 14:08:36 +0200, Fiona Ebner wrote: > After mirror, the node below throttle might not be the format node, > but can also be a zeroinit node. In particular, this is the node that > needs to be used by when replacing the blockdev for volume-chain > snapshots for the 'current' snaps

Re: [pve-devel] [PATCH qemu-server] fix #6207: vm status: cache last disk read/write values

2025-09-22 Thread Thomas Lamprecht
Am 22.09.25 um 12:18 schrieb Fiona Ebner: > If disk read/write cannot be queried because of QMP timeout, they > should not be reported as 0, but the last value should be re-used. > Otherwise, the difference between that reported 0 and the next value, > when the stats are queried successfully, will

Re: [pve-devel] [PATCH pve-manager 4/5] api: add service state return schema

2025-09-22 Thread Thomas Lamprecht
Am 19.09.25 um 11:40 schrieb n.f...@proxmox.com: > From: Nicolas Frey > > Signed-off-by: Nicolas Frey > --- > PVE/API2/Services.pm | 30 +- > 1 file changed, 29 insertions(+), 1 deletion(-) > > diff --git a/PVE/API2/Services.pm b/PVE/API2/Services.pm > index 708b661

[pve-devel] applied: [PATCH pve-manager 1/5] Fix grammar in ifupdown2 version error message

2025-09-22 Thread Thomas Lamprecht
Am 19.09.25 um 11:40 schrieb n.f...@proxmox.com: > From: Nicolas Frey > > Signed-off-by: Nicolas Frey > --- > PVE/API2/Network.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > applied this one already, thanks! ___ pve-devel mailing l

Re: [pve-devel] [PATCH pve-manager 2/5] api: add ACME plugin return schema

2025-09-22 Thread Thomas Lamprecht
Am 19.09.25 um 11:40 schrieb n.f...@proxmox.com: > From: Nicolas Frey > FYI: these methods are operating on modules that base off a so called SectionConfig. This is quite widely used in our stack, not only in PVE but also in rust based projects like PBS. I'd recommend skimming the SectionConfig

Re: [pve-devel] [PATCH V3 pve-storage] lvm: use blkdiscard instead cstream to saferemove drive

2025-09-22 Thread Fiona Ebner
Am 18.08.25 um 2:09 PM schrieb Alexandre Derumier via pve-devel: > Current cstream implementation is pretty slow as hell, even without > throttling. I'd use either "pretty slow" or "slow as hell" > use blkdiscard --zeroout instead, which is lot of magnetude faster should be: "a few magnitudes"

[pve-devel] [PATCH installer v3 0/3] remove mirror selection from installer

2025-09-22 Thread Shannon Sterz
these three patches remove the debian mirror selection from the installer as it is no longer needed. debian recommends using its CDN these days [1]. so we don't need to select a mirror. [1]: https://www.debian.org/releases/trixie/release-notes/upgrading.en.html#adding-apt-internet-sources Change

[pve-devel] [PATCH installer v3 3/3] html/country: update explanatory text to not mention setting a mirror

2025-09-22 Thread Shannon Sterz
as we don't set one anymore, but instead just use debian's CDN. Suggested-by: Fiona Ebner Signed-off-by: Shannon Sterz --- html/country.htm | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/html/country.htm b/html/country.htm index 990a39f..1271226 100644 --- a/html/c

[pve-devel] [PATCH proxmox-acme 0/1] add all missing dns challenge schemas

2025-09-22 Thread n . frey
From: Nicolas Frey Added all missing DNS challenge schemas in accordance with acme.sh dnsapi [0] wiki. [0] https://github.com/acmesh-official/acme.sh/wiki/dnsapi Nicolas Frey (1): add all missing dns challenge schemas src/dns-challenge-schema.json | 2077 + 1

[pve-devel] [PATCH proxmox-acme 1/1] add all missing dns challenge schemas

2025-09-22 Thread n . frey
From: Nicolas Frey Signed-off-by: Nicolas Frey --- src/dns-challenge-schema.json | 2077 + 1 file changed, 1864 insertions(+), 213 deletions(-) diff --git a/src/dns-challenge-schema.json b/src/dns-challenge-schema.json index 8669c86..4d69e68 100644 --- a/src/dns

[pve-devel] [PATCH installer v3 2/3] country.pl: remove generating mirror information

2025-09-22 Thread Shannon Sterz
this is no longer needed, as we don't rely on this information in the installer anymore. Suggested-by: Fiona Ebner Signed-off-by: Shannon Sterz --- country.pl| 12 +- .../tests/resources/locales.json | 249 -- 2 files changed, 2

Re: [pve-devel] [PATCH installer v2 2/3] country.pl: remove generating mirror information

2025-09-22 Thread Shannon Sterz
On Mon Sep 22, 2025 at 12:33 PM CEST, Christoph Heiss wrote: > On Mon Sep 22, 2025 at 11:08 AM CEST, Shannon Sterz wrote: >> this is no longer needed, as we don't rely on this information in the >> installer anymore. > > `proxmox-auto-installer/tests/resources/locales.json` can/should also be > upd

[pve-devel] [PATCH ha-manager 2/5] build: install PVE::HA::Rules::* in simulator as well

2025-09-22 Thread Fabian Grünbichler
Signed-off-by: Fabian Grünbichler --- src/PVE/HA/Makefile | 1 + src/PVE/HA/Rules/Makefile | 8 +++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/src/PVE/HA/Makefile b/src/PVE/HA/Makefile index 8862907..f3f09a8 100644 --- a/src/PVE/HA/Makefile +++ b/src/PVE/HA/Makefile

[pve-devel] [PATCH ha-manager 5/5] sim: add Cluster.pm stub with completion stub

2025-09-22 Thread Fabian Grünbichler
Signed-off-by: Fabian Grünbichler --- the same applies here - instead of the stub, we could also conditionally require PVE::Cluster in NodeAffinity.pm .. src/PVE/HA/Sim/Cluster.pm | 11 +++ src/PVE/HA/Sim/Makefile | 2 ++ 2 files changed, 13 insertions(+) create mode 100644 src/PVE/H

[pve-devel] [PATCH ha-manager 1/5] fix #6839: move PVE::Notify usage to "real" Env

2025-09-22 Thread Fabian Grünbichler
instead of doing it in PVE::NodeStatus, which is also pulled into the simulator. Signed-off-by: Fabian Grünbichler --- alternatively, we could also use the $have_.. / require hack here? src/PVE/HA/Env/PVE2.pm | 5 + src/PVE/HA/NodeStatus.pm | 4 +--- 2 files changed, 6 insertions(+), 3 de

[pve-devel] [PATCH ha-manager 4/5] build: install PVE::HA::Usage::Basic in simulator as well

2025-09-22 Thread Fabian Grünbichler
only install Basic module, as the Static one has additional dependencies Signed-off-by: Fabian Grünbichler --- src/PVE/HA/Makefile | 1 + src/PVE/HA/Usage/Makefile | 8 +++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/src/PVE/HA/Makefile b/src/PVE/HA/Makefile index f3

[pve-devel] [RFC ha-manager 0/5] make simulator standalone again

2025-09-22 Thread Fabian Grünbichler
by moving PVE::Notify to the actual site where it is used, and adapting the build parts to install PVE::HA::Usage::Basic and PVE::HA::Rules::* in the simulator as well. CRS is not supported for now, as that would require pulling in a lot more parts of PVE into the dependency tree of the simulator.

[pve-devel] [PATCH pve-network v6 2/2] ipam: add test cases for nautobot plugin

2025-09-22 Thread Lou Lecrivain via pve-devel
--- Begin Message --- From: Lou Lecrivain Co-authored-by: Lou Lecrivain Signed-off-by: Hannes Duerr Signed-off-by: Lou Lecrivain --- src/test/ipams/nautobot/expected.add_ip | 11 + .../ipams/nautobot/expected.add_ip_notgateway | 11 + .../ipams/nautobot/expected.add_next

[pve-devel] [PATCH qemu-server] fix #6207: vm status: cache last disk read/write values

2025-09-22 Thread Fiona Ebner
If disk read/write cannot be queried because of QMP timeout, they should not be reported as 0, but the last value should be re-used. Otherwise, the difference between that reported 0 and the next value, when the stats are queried successfully, will show up as a huge spike in the RRD graphs. Invali

[pve-devel] [PATCH pve-network v6 1/2] ipam: add Nautobot plugin

2025-09-22 Thread Lou Lecrivain via pve-devel
--- Begin Message --- From: Lou Lecrivain Nautobot is a network source of truth [0] and as such this plugin offers the possibility to automatically enter IP addresses and subnets (prefixes in Nautobot jargon) into Nautobot as soon as they are created in Proxmox VE. Limitations: * Only Nautobot v

[pve-devel] [PATCH i18n] fix Czech translation of "Order Certificates Now"

2025-09-22 Thread Filip Schauer
The previous translation incorrectly suggested that "Order Certificates Now" means sorting the certificates. Signed-off-by: Filip Schauer --- Spotted while checking the translations into my second mother tongue in Proxmox Mail Gateway. cs.po | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[pve-devel] [PATCH installer v2 3/3] html/country: update explanatory text to not mention setting a mirror

2025-09-22 Thread Shannon Sterz
as we don't set one anymore, but instead just use debian's CDN. Suggested-by: Fiona Ebner Signed-off-by: Shannon Sterz --- html/country.htm | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/html/country.htm b/html/country.htm index 990a39f..1271226 100644 --- a/html/c

[pve-devel] [PATCH pve-docs v6] add documentation for nautobot ipam plugin

2025-09-22 Thread Lou Lecrivain via pve-devel
--- Begin Message --- From: lou lecrivain Co-authored-by: Jonatan Crystall Co-authored-by: Lou Lecrivain Signed-off-by: Hannes Duerr Signed-off-by: Lou Lecrivain --- pvesdn.adoc | 20 1 file changed, 20 insertions(+) diff --git a/pvesdn.adoc b/pvesdn.adoc index 8759691.

[pve-devel] [PATCH pve-manager v6] ipam: add UI dialog for nautobot ipam plugin

2025-09-22 Thread Lou Lecrivain via pve-devel
--- Begin Message --- From: Lou Lecrivain Co-authored-by: Jonatan Crystall Co-authored-by: Lou Lecrivain Signed-off-by: Hannes Duerr Signed-off-by: Lou Lecrivain --- www/manager6/Makefile | 1 + www/manager6/Utils.js | 5 +++ www/manager6/sdn/ipams/Nautobo

[pve-devel] Superseded: Re: [PATCH installer v2] install: don't select a debian mirror based on the country anymore

2025-09-22 Thread Shannon Sterz
On Mon Sep 22, 2025 at 10:56 AM CEST, Fiona Ebner wrote: > Am 22.09.25 um 10:35 AM schrieb Shannon Sterz: >> On Wed Sep 17, 2025 at 3:38 PM CEST, Fiona Ebner wrote: >>> Am 31.07.25 um 11:15 AM schrieb Shannon Sterz: newer isos ship with new deb822 style repository configurations in `/etc/

[pve-devel] [PATCH installer v2 1/3] install: don't select a debian mirror based on the country anymore

2025-09-22 Thread Shannon Sterz
newer isos ship with new deb822 style repository configurations in `/etc/apt/sources.list.d/debian.sources` which already use the now prefered cdn [1]. since setting a mirror like this does not work anymore due to this change, just remove it to keep using the prefered cdn. [1]: https://www.debian.

[pve-devel] [PATCH installer v2 2/3] country.pl: remove generating mirror information

2025-09-22 Thread Shannon Sterz
this is no longer needed, as we don't rely on this information in the installer anymore. Suggested-by: Fiona Ebner Signed-off-by: Shannon Sterz --- country.pl | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/country.pl b/country.pl index 227eba4..be760bb 100755

Re: [pve-devel] [PATCH installer v2] install: don't select a debian mirror based on the country anymore

2025-09-22 Thread Fiona Ebner
Am 22.09.25 um 10:35 AM schrieb Shannon Sterz: > On Wed Sep 17, 2025 at 3:38 PM CEST, Fiona Ebner wrote: >> Am 31.07.25 um 11:15 AM schrieb Shannon Sterz: >>> newer isos ship with new deb822 style repository configurations in >>> `/etc/apt/sources.list.d/debian.sources` which already use the now >>