Re: [pve-devel] [PATCH storage/qemu-server/manager v6] implement ova/ovf import for file based storages

2024-11-21 Thread Fabian Grünbichler
> Gilberto Ferreira via pve-devel hat am > 21.11.2024 13:23 CET geschrieben: > By the way, let me take the opportunity to ask a question: Are there any > plans to make this import tool accept other image formats like raw, qcow2, > vmdk or vhd(x) without OVA format? > Sometimes, all we have is a

[pve-devel] Proxmox VE 8.3 released!

2024-11-21 Thread Thomas Lamprecht
Hi All! We are excited to announce that our latest software version 8.3 for Proxmox Virtual Environment is now available for download. This release is based on Debian 12.8 "Bookworm" but uses a newer Linux kernel 6.8.12-4 and kernel 6.11 as opt-in, QEMU 9.0.2, LXC 6.0.0, and ZFS 2.2.6 (with compat

Re: [pve-devel] [PATCH v1 pve-common 10/18] pbsclient: use `File::Spec->catfile` to concatenate file paths

2024-11-21 Thread Max Carrara
On Mon Nov 11, 2024 at 8:16 PM CET, Thomas Lamprecht wrote: > a commit message explaining why/benefits would be good – especially as IIRC > we currently do not use this module at all? Sure, I'll add one! I've opted to using `File::Spec` here because I've been using it a lot for other things (that

Re: [pve-devel] [PATCH storage/qemu-server/manager v6] implement ova/ovf import for file based storages

2024-11-21 Thread Dominik Csapak
On 11/20/24 17:57, Gilberto Ferreira via pve-devel wrote: Ok. Here is the error message: referenced path 'Ubuntu 24.04 (64bit)-disk001.vmdk' is invalid (500) hi, it's probably the parenthesis in the filename. would you mind opening a bug in our bugzilla so we can keep track of that? if you

Re: [pve-devel] applied: [PATCH proxmox-i18n] updated Japanese language ja.po

2024-11-21 Thread ribbon via pve-devel
--- Begin Message --- On 2024/11/21 0:58, Thomas Lamprecht wrote: Am 19.11.24 um 17:32 schrieb Seiji Makino: Fixed Japanese translation. Signed-off-by: Seiji Makino --- ja.po | 408 ++ 1 file changed, 183 insertions(+), 225 deletions(-

[pve-devel] [PATCH proxmox-i18n] es: Update a Start Address translation

2024-11-21 Thread Maximiliano Sandoval
This is a very small correction which matches better the context. Signed-off-by: Maximiliano Sandoval --- es.po | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/es.po b/es.po index edb20c0..c6d04fe 100644 --- a/es.po +++ b/es.po @@ -8,7 +8,7 @@ msgstr "" "Project-Id-Versi

Re: [pve-devel] [PATCH storage/qemu-server/manager v6] implement ova/ovf import for file based storages

2024-11-21 Thread Gilberto Ferreira via pve-devel
--- Begin Message --- By the way, let me take the opportunity to ask a question: Are there any plans to make this import tool accept other image formats like raw, qcow2, vmdk or vhd(x) without OVA format? Sometimes, all we have is a VHD file, generated by disk2vhd, for example. That would be great.

Re: [pve-devel] Proposal: support for atomic snapshot of all VM disks at once

2024-11-21 Thread Max Carrara
On Wed Nov 20, 2024 at 5:10 PM CET, Ivaylo Markov wrote: > Hello, > > I've been caught up in other things and it's been a while, but as I was > collating and testing my proposed changes, I came across this again and > thought I'd clarify something. > > On 08/10/2024 13:50, Max Carrara wrote: > >>

Re: [pve-devel] [PATCH storage/qemu-server/manager v6] implement ova/ovf import for file based storages

2024-11-21 Thread Gilberto Ferreira via pve-devel
--- Begin Message --- Never mind. I tried to import via VirtualBox and got the same result. I guess the ova from that web site has some malformed files. But eventually I will open a bug report, if I encounter further issues. Thanks Cheers --- Em qui., 21 de nov. de 2024 às 05:24, Dominik Csa

Re: [pve-devel] [PATCH] webhook edit: do not split translatable strings

2024-11-21 Thread Shannon Sterz
On Thu Nov 21, 2024 at 3:28 PM CET, Maximiliano Sandoval wrote: > Splitting translatable strings deprives translators for context which > might be needed for getting the correct tenses or genders. See [1] for > example. > > [1] > https://wiki.gnome.org/TranslationProject(2f)DevGuidelines(2f)Never(

[pve-devel] [PATCH qemu-server v2] fix #3588: helper: consider NIC count for config-specific timeout

2024-11-21 Thread Hannes Laimer
There have been some reports about `qm start` timeouts on VMs that have a lot of NICs assigned. This patch considers the number of NICs when calculating the config-specific timeout. Since the increase in start time is linearly related to the number of NICs, a constant timeout increment per NIC was

Re: [pve-devel] [PATCH qemu-server] fix #3588: helper: consider NIC count for config-specific timeout

2024-11-21 Thread Hannes Laimer
sent a v2 [1] [1] https://lore.proxmox.com/pve-devel/20241121151335.130711-1-h.lai...@proxmox.com/T/#u On 11/15/24 11:00, Hannes Laimer wrote: There have been some reports about `qm start` timeouts on VMs that have a lot of NICs assigned. This patch considers the number of NICs when calculati

Re: [pve-devel] [PATCH v1 pve-common 08/18] pbsclient: document package and its public functions & methods

2024-11-21 Thread Max Carrara
On Mon Nov 11, 2024 at 8:14 PM CET, Thomas Lamprecht wrote: > Am 02.08.24 um 15:26 schrieb Max Carrara: > > This commit adds a brief overview for the `PVE::PBSClient` package and > > documents its public functions and methods. Examples are added where > > deemed appropriate. > > great works and tha

[pve-devel] [PATCH] webhook edit: do not split translatable strings

2024-11-21 Thread Maximiliano Sandoval
Splitting translatable strings deprives translators for context which might be needed for getting the correct tenses or genders. See [1] for example. [1] https://wiki.gnome.org/TranslationProject(2f)DevGuidelines(2f)Never(20)split(20)sentences.html Signed-off-by: Maximiliano Sandoval --- src/p

Re: [pve-devel] [PATCH qemu-server] fix #3588: helper: consider NIC count for config-specific timeout

2024-11-21 Thread Fiona Ebner
Am 15.11.24 um 11:00 schrieb Hannes Laimer: > There have been some reports about `qm start` timeouts on VMs that have a > lot of NICs assigned. > This patch considers the number of NICs when calculating the config-specific > timeout. Since the increase in start time is linearly related to the numbe

[pve-devel] [PATCH proxmox-ve-rs 3/3] d/control: include a revision in the package version

2024-11-21 Thread Jing Luo via pve-devel
--- Begin Message --- The "dsc" target also fails because of this: error: can't build with source format '3.0 (quilt)': non-native package version does not contain a revision d/source/format is automatically generated as '3.0 (quilt)', let's keep it that way. Include a dch entry so no one will

[pve-devel] [PATCH proxmox-ve-rs 2/3] d/control: specify the build-deps not needed for "nocheck" profile

2024-11-21 Thread Jing Luo via pve-devel
--- Begin Message --- These rust libs are not needed if using DEB_BUILD_PROFILES="nocheck". This helps bootstrap proxmox to another arch in the future. Signed-off-by: Jing Luo --- proxmox-ve-config/debian/control | 32 1 file changed, 16 insertions(+), 16 deletio

[pve-devel] [PATCH proxmox-ve-rs 1/3] d/control: correct source package name

2024-11-21 Thread Jing Luo via pve-devel
--- Begin Message --- Currently the "dsc" target fails and dpkg-buildpackage complains: error: can't build with source format '3.0 (quilt)': no upstream tarball found at ../proxmox-ve-config_0.2.0.orig.tar.{bz2,gz,lzma,xz} but there is a "rust-proxmox-ve-config_0.2.0.orig.tar.gz". Usually a rust

Re: [pve-devel] [PATCH v1 pve-common 09/18] pbsclient: create secret dir with `mkdir -p` and mode `700`

2024-11-21 Thread Max Carrara
On Mon Nov 11, 2024 at 8:16 PM CET, Thomas Lamprecht wrote: > Am 02.08.24 um 15:26 schrieb Max Carrara: > > .. instead of using a regular `mkdir` call. > > > > The `File::Path::make_path` subroutine is used for this purpose, which > > recursively creates all directories if they didn't exist before

Re: [pve-devel] [PATCH v1 pve-common 11/18] pbsclient: let `status` method return a hash instead of an array

2024-11-21 Thread Max Carrara
On Mon Nov 11, 2024 at 8:17 PM CET, Thomas Lamprecht wrote: > Am 02.08.24 um 15:26 schrieb Max Carrara: > > Instead of returning an (arguably awkward) array with four elements, > > can be fine, but FWIW, this is how our storage plugin system expects the > status, but currently that calls run_client