[pve-devel] applied: [PATCH qemu-server] tests: add cache && aio tests for differents storage type

2025-06-11 Thread Fiona Ebner
Applied, thanks! I renamed the volume group to avoid using the same name as the already existing lvm-thin storage. I also had already written a test for this with a single config and applied that too, as it better covers the local storage case. [1/1] tests: add cache && aio tests for differents st

Re: [pve-devel] [PATCH widget-toolkit v2] window: ACMEDomains: show error dialog on domain limit

2025-06-11 Thread Christoph Heiss
Ping, still applies. On Tue Mar 4, 2025 at 12:57 PM CET, Christoph Heiss wrote: > Currently, this just fails silently for users and only logs an error to > the console. Improve the error message & show an alert dialog on > failure. > > This affects both PMG & PBS, as they use the exact same codepa

Re: [pve-devel] [PATCH qemu-server v2 7/13] fix #5180: libexec: add QEMU dbus-vmstate daemon for migrating conntrack

2025-06-11 Thread Christoph Heiss
On Tue Jun 3, 2025 at 11:34 AM CEST, Stefan Hanreich wrote: > On 4/24/25 13:19, Christoph Heiss wrote: >> +sub Load { >> +my ($self, $bytes) = @_; >> + >> +my $len = scalar(@$bytes); >> +return if $len <= 1; # see also the `Save` method >> + >> +my $text = pack('c*', @$bytes); >> +

[pve-devel] [PATCH proxmox-i18n v3] make: add proxmox-datacenter-manager translations

2025-06-11 Thread Maximiliano Sandoval
The catalog-{lang}.mo files are generated only with strings that are relevant to the proxmox-datacenter-manager instead of the whole {lang}.po file. The msgmerge command will produce all strings containing in the {lang}.po file but the ones that do not concern proxmox-datacenter-manager.pot will be

Re: [pve-devel] [PATCH proxmox-i18n v2 2/2] make: add proxmox-datacenter-manager translations

2025-06-11 Thread Maximiliano Sandoval
Maximiliano Sandoval writes: > > The requirements are: > > - > https://lore.proxmox.com/pdm-devel/20250122150132.426276-1-m.sando...@proxmox.com/ > - > https://lore.proxmox.com/yew-devel/20250127094651.139204-1-m.sando...@proxmox.com/ > > > ___ > pv

[pve-devel] [PATCH docs] sdn: add fingerprint option to the NetBox IPAM plugin

2025-06-11 Thread Gabriel Goller
Add the required "Fingerprint" option of the NetBox IPAM plugin to the docs. Also add a quick `openssl` command to retrieve the fingerprint from the default certificate on a NetBox installation. AFAIK NetBox doesn't provide a way to easily get the fingerprint from the gui. Signed-off-by: Gabriel G

Re: [pve-devel] [PATCH-SERIES v6 pve-storage/qemu-server/pve-qemu] add external qcow2 snapshot support

2025-06-11 Thread joao.sousa via pve-devel
--- Begin Message --- On 10/06/25 10:54, Fiona Ebner wrote: > What exactly is your goal with the feature? If you want to build > something on top of it, it might be better to wait until it is > finalized/applied. Currently, the most helpful contribution with the > feature itself would be testing an

Re: [pve-devel] [PATCH cluster-pve8 2/2] status: handle new pve9- metrics update data

2025-06-11 Thread Aaron Lauterer
On 2025-06-02 15:31, Thomas Lamprecht wrote: Am 23.05.25 um 18:00 schrieb Aaron Lauterer: For PVE9 there will be additional fields in the metrics that are collected. The new columns/fields are added at the end of the current ones. Therefore, if we get the new format, we need to cut it. Pat

Re: [pve-devel] [PATCH qemu-server 01/13] blockdev: cmdline: add blockdev syntax support

2025-06-11 Thread DERUMIER, Alexandre via pve-devel
--- Begin Message --- > > > >> > >I still unsure how to handle same volume multiple time (if we really > >want it). I was thinking to use the deviceid in the name > >(virtio0,..), > >but it don't work when you unplug/replug to a deviceid. >>Why wouldn't it work? Ah sorry, I forgot that we delete

Re: [pve-devel] [PATCH qemu-server 01/13] blockdev: cmdline: add blockdev syntax support

2025-06-11 Thread Fiona Ebner
Am 11.06.25 um 16:02 schrieb DERUMIER, Alexandre: > (I'll be busy with proxmox training tomorrow && friday, so I'll rework > on it next week) I did mention in an earlier mail that I'm already working on a series for the switch to blockdev based on your patches myself. Sorry, if that wasn't clear!

[pve-devel] [PATCH manager 2/2] ui: one-shot backup: remove 'auto' notification mode for clarity

2025-06-11 Thread Lukas Wagner
The 'auto' mode does not really add any functionality but only adds confusion about what it actually does, so we completely remove it from the UI. It is still supported by the backend, but in the UI we map it to a concrete mode (either notification-system or legacy-sendmail, depending on whether ma

[pve-devel] [PATCH manager 1/2] ui: backup job edit: move notification related settings to separate tab

2025-06-11 Thread Lukas Wagner
The notification settings in the 'General' tab were unfortunately a source of regular confusion for many people. This was primarily due to the behavior of the 'motification mode'. The notification mode can one of the following: - notification-system: Emit a notification event to the global no

[pve-devel] [PATCH container v2 07/11] setup: debian: create /etc/network path if missing

2025-06-11 Thread Filip Schauer
This prevents an error during Debian container setup when the /etc/network directory is missing. This fixes container creation from Debian based OCI images. Signed-off-by: Filip Schauer --- src/PVE/LXC/Setup/Debian.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/src/PVE/LXC/Setup/Debian.p

[pve-devel] [PATCH container v2 08/11] setup: recursively mkdir /etc/systemd/{network, system-preset}

2025-06-11 Thread Filip Schauer
Ensure that both /etc/systemd/network and /etc/systemd/system-preset exist before writing files into them. This fixes container creation from the docker.io/fedora & docker.io/ubuntu OCI images. Signed-off-by: Filip Schauer --- src/PVE/LXC/Setup/Base.pm | 3 ++- 1 file changed, 2 insertions(+), 1

[pve-devel] [PATCH container v2 06/11] configure static IP in LXC config for custom entrypoint

2025-06-11 Thread Filip Schauer
When a container uses the default `/sbin/init` entrypoint, network interface configuration is usually managed by processes within the container. However, containers with a different entrypoint might not have any internal network management process. Consequently, IP addresses might not be assigned.

[pve-devel] [PATCH container v2 04/11] add support for OCI images as container templates

2025-06-11 Thread Filip Schauer
This aims to add basic support for the Open Container Initiative image format according to the specification. [0] [0] https://github.com/opencontainers/image-spec/blob/main/spec.md Signed-off-by: Filip Schauer --- src/PVE/API2/LXC.pm | 64 ++--- 1 file ch

[pve-devel] [PATCH manager v2 11/11] ui: storage upload: accept *.tar files as vztmpl

2025-06-11 Thread Filip Schauer
Signed-off-by: Filip Schauer --- www/manager6/window/UploadToStorage.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/window/UploadToStorage.js b/www/manager6/window/UploadToStorage.js index f6cad0ec..cfd0a949 100644 --- a/www/manager6/window/UploadToStorage.js

[pve-devel] [PATCH container/manager/proxmox{, -perl-rs}/storage v2 00/11] support OCI images as container templates

2025-06-11 Thread Filip Schauer
Add basic support for OCI (Open Container Initiative) images [0] as container templates. An OCI image can be for example obtained from Docker Hub: Either using Docker: ``` $ docker pull httpd $ docker save httpd > httpd.tar ``` Or using Podman: When using Podman, the format needs to be explicit

[pve-devel] [PATCH container v2 05/11] config: add entrypoint parameter

2025-06-11 Thread Filip Schauer
Signed-off-by: Filip Schauer --- src/PVE/API2/LXC.pm | 2 +- src/PVE/LXC.pm| 2 ++ src/PVE/LXC/Config.pm | 12 3 files changed, 15 insertions(+), 1 deletion(-) diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm index 3b0ea35..cf93778 100644 --- a/src/PVE/API2/LXC.pm

[pve-devel] [PATCH proxmox-perl-rs v2 02/11] add Perl mapping for OCI container image parser/extractor

2025-06-11 Thread Filip Schauer
Signed-off-by: Filip Schauer --- pve-rs/Cargo.toml | 2 ++ pve-rs/Makefile | 1 + pve-rs/debian/control | 2 ++ pve-rs/src/lib.rs | 1 + pve-rs/src/oci.rs | 20 5 files changed, 26 insertions(+) create mode 100644 pve-rs/src/oci.rs diff --git a/pve-rs/

[pve-devel] [PATCH proxmox v2 01/11] add proxmox-oci crate

2025-06-11 Thread Filip Schauer
This crate can parse and extract an OCI image bundled as a tar archive. Signed-off-by: Filip Schauer --- Cargo.toml | 1 + proxmox-oci/Cargo.toml | 22 proxmox-oci/debian/changelog | 5 + proxmox-oci/debian/control | 47 proxmox-oci/d

[pve-devel] [PATCH container v2 09/11] manage DHCP for containers with custom entrypoint

2025-06-11 Thread Filip Schauer
Containers that do not use the default `/sbin/init` entrypoint may lack in‑container network management. A previous commit already handles static IP addresses. Now this commit also handles DHCP. This is done using a `dhclient` process for each network interface. Signed-off-by: Filip Schauer ---

[pve-devel] [PATCH storage v2 10/11] allow .tar container templates

2025-06-11 Thread Filip Schauer
This is needed for OCI container images bundled as tar files, as generated by `docker save`. OCI images do not need additional compression, since the content is usually compressed already. Signed-off-by: Filip Schauer --- src/PVE/Storage.pm| 2 +- src/PVE/Storage/Plugin.pm | 2 +- 2 file

Re: [pve-devel] [PATCH container/proxmox{, -perl-rs}/storage 0/9] support OCI images as container templates

2025-06-11 Thread Filip Schauer
On 02/06/2025 18:26, Michael Köppl wrote: Thanks for tackling this! I tested setting up containers based on various OCI images. Apart from the UI not allowing upload of .tar files (see my comment on the pve-storage patch), the uploading worked as expected. I encountered some problems with various

Re: [pve-devel] [PATCH container/proxmox{, -perl-rs}/storage 0/9] support OCI images as container templates

2025-06-11 Thread Filip Schauer
On 06/06/2025 15:19, Christoph Heiss wrote: The nginx image, both -alpine and -bookworm, fail to start with: lxc-info: 103: ../src/lxc/af_unix.c: lxc_abstract_unix_recv_fds_iov: 218 Connection reset by peer - Failed to receive response lxc-info: 103: ../src/lxc/commands.c: lxc_cmd_rsp_recv_fds:

Re: [pve-devel] superseded: [PATCH container/proxmox{, -perl-rs}/storage 0/9] support OCI images as container templates

2025-06-11 Thread Filip Schauer
Superseded by: https://lore.proxmox.com/pve-devel/20250611144903.200940-1-f.scha...@proxmox.com/ On 20/05/2025 14:42, Filip Schauer wrote: Add basic support for OCI (Open Container Initiative) images [0] as container templates. ___ pve-devel mailing

Re: [pve-devel] [PATCH common 4/4] metrics: add buffer and cache to meminfo

2025-06-11 Thread Aaron Lauterer
On 2025-06-02 16:07, Thomas Lamprecht wrote: Am 23.05.25 um 18:37 schrieb Aaron Lauterer: From: Folke Gleumes Expose buffers and cache as separate metrics instead of including them in memfree and memused. Originally-by: Folke Gleumes [AL: rebased and adapted to changes that happened in

Re: [pve-devel] Plans for Realtime Kernel and Realtime Configuration Support in Proxmox?

2025-06-11 Thread Fabian Grünbichler
> Florent Carli via pve-devel hat am 06.06.2025 > 10:22 CEST geschrieben: > Hello, > > After trying the user forum first, I'm turning to you here. I did a > quick Google search to make sure the question hadn't already been > answered (apparently not). Now I'm reaching out to the real experts :)

[pve-devel] [PATCH container v2 03/11] config: whitelist lxc.init.cwd

2025-06-11 Thread Filip Schauer
This parameter allows setting the working directory of the init process in the container. This can be used by containers created from an OCI image, that specifies a custom working directory. Signed-off-by: Filip Schauer --- src/PVE/LXC/Config.pm | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [pve-devel] [PATCH qemu-server 01/13] blockdev: cmdline: add blockdev syntax support

2025-06-11 Thread Fiona Ebner
Am 11.06.25 um 08:37 schrieb DERUMIER, Alexandre: >>> Also, while the use case here shouldn't be cryptographically >>> sensitive, >>> you never know, so I'll just use a different hash function than sha1. >>> I'll cut off the result from that hash to 30 hex digits. Then we >>> still >>> have one let

Re: [pve-devel] [PATCH qemu-server 01/13] blockdev: cmdline: add blockdev syntax support

2025-06-11 Thread DERUMIER, Alexandre via pve-devel
--- Begin Message --- Am 11.06.25 um 16:02 schrieb DERUMIER, Alexandre: > (I'll be busy with proxmox training tomorrow && friday, so I'll > rework > on it next week) >>I did mention in an earlier mail that I'm already working on a series >>for the switch to blockdev based on your patches myself.