Re: [pve-devel] [PATCH proxmox-perl-rs] update to proxmox-log 0.2

2024-08-06 Thread Lukas Wagner
On 2024-07-25 12:56, Fabian Grünbichler wrote: > Signed-off-by: Fabian Grünbichler > --- > common/src/logger.rs | 4 +--- > pmg-rs/Cargo.toml | 2 +- > pmg-rs/debian/control | 2 +- > pve-rs/Cargo.toml | 2 +- > pve-rs/debian/control | 2 +- > 5 files changed, 5 insertions(+), 7 delet

[pve-devel] applied: [PATCH proxmox-perl-rs] update to proxmox-log 0.2

2024-08-06 Thread Fabian Grünbichler
with your R-B, thanks! Quoting Lukas Wagner (2024-08-06 13:07:03) > > > On 2024-07-25 12:56, Fabian Grünbichler wrote: > > Signed-off-by: Fabian Grünbichler > > --- > > common/src/logger.rs | 4 +--- > > pmg-rs/Cargo.toml | 2 +- > > pmg-rs/debian/control | 2 +- > > pve-rs/Cargo.toml

[pve-devel] [PATCH common 2/2] SysFSTools: lscpi: move mdev and iommugroup check outside of verbose

2024-08-06 Thread Dominik Csapak
they should not be expensive (only reading/file checking in sysfs; the parsed vendor/id names are not required) so we should include them always. We need at least the mdev part later at a point where we're not interested in the rest of the verbose mode. Signed-off-by: Dominik Csapak --- best vie

[pve-devel] [PATCH common 1/2] SysFSTools: handle new nvidia syfsapi as mdev

2024-08-06 Thread Dominik Csapak
with kernel 6.8 NVIDIAs vGPU driver has a different api than the previous 'mediated devices'. Adapt our sysfcode to also recognize this for the 'mdev' paths and add another 'nvidia' property so we can detect this. Also parse the new api when they exist instead of the mediated devices. The biggest

[pve-devel] [PATCH common/qemu-server/manager] adapt to nvidia vgpu api changes

2024-08-06 Thread Dominik Csapak
For many new cards, nvidia changed the kernel interface since kernel verion 6.8. Instead of using mediated devices, they provide their own api. This series adapts to that, with no required change to the vm config, and only minimal changes to our api. The biggest change is that the mdev types can

[pve-devel] [PATCH manager 1/1] api/ui: improve mdev listing for pci mappings

2024-08-06 Thread Dominik Csapak
currently when we have a pci resource mapping, we manually check only the available models for the first pci entry. This often works, but not always, since one could have completely different devices in one mapping, or with the new nvidia sysfs api we don't get the generally available models. To i

[pve-devel] [PATCH qemu-server 2/3] pci: remove pci reservation: optionally give list of ids to remove

2024-08-06 Thread Dominik Csapak
so we can partially release ids again. This will be necessary for NVIDIAs new sysfs api Signed-off-by: Dominik Csapak --- PVE/QemuServer/PCI.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/PVE/QemuServer/PCI.pm b/PVE/QemuServer/PCI.pm index 97eb2165..ae180e08 100644 ---

[pve-devel] [PATCH qemu-server 1/3] pci: choose devices: don't reserve pciids when vm is already running

2024-08-06 Thread Dominik Csapak
since the only way this could happen is when we're being called from 'qm showcmd' and there we don't want to reserve or create anything. In case the vm was not running, we actually reserve the devices, so we want to call 'cleanup_pci_devices' after to remove those again. This minimizes the timespa

[pve-devel] [PATCH qemu-server 3/3] pci: mdev: adapt to nvidia interface with kernel >= 6.8

2024-08-06 Thread Dominik Csapak
since kernel 6.8, NVIDIAs vGPU driver does not use the generic mdev interface anymore, since they relied on a feature there which is not available anymore. IIUC the kernel [0] recommends drivers to implement their own device specific features since putting all in the generic one does not make sense

[pve-devel] [PATCH novnc 2/2] bump version to 1.5.0-1

2024-08-06 Thread Dominik Csapak
Signed-off-by: Dominik Csapak --- debian/changelog | 6 ++ 1 file changed, 6 insertions(+) diff --git a/debian/changelog b/debian/changelog index a21138a..a911754 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,9 @@ +novnc-pve (1.5.0-1) bookworm; urgency=medium + + * update

[pve-devel] [PATCH novnc 1/2] upgrade noVNC and patches to 1.5.0

2024-08-06 Thread Dominik Csapak
Signed-off-by: Dominik Csapak --- .../0001-add-PVE-specific-JS-code.patch | 4 ++-- ...002-add-custom-fbresize-event-on-rfb.patch | 10 +- ...nge-scaling-when-toggling-fullscreen.patch | 6 +++--- ...rectory-for-fetching-images-js-files.patch | 20 +-- .../0009-dec

Re: [pve-devel] [PATCH storage] storage: add bzip2 support

2024-08-06 Thread Maximiliano Sandoval
Stoiko Ivanov writes: > In general I think the addition of bz2 for ISOs makes sense (if only for > OPNSense's sake) - Thanks for addressing this! > > You want to add bzip2 as dependency to debian/control > we probably should also add lzop, zstd, gzip to the dependencies > although they are pulle

[pve-devel] [PATCH storage v2 2/2] d/control: dependencies: add bzip2, gzip, lzpo, zstd

2024-08-06 Thread Maximiliano Sandoval
The decompressor_info method calls binaries provided by these packages so they are (alphabetically) added explicitly as dependencies. To avoid a build-time error E: libpve-storage-perl: depends-on-essential-package-without-using-version Depends: gzip the current minor version available in b

[pve-devel] [PATCH storage v2 1/2] storage: add bzip2 support

2024-08-06 Thread Maximiliano Sandoval
Support for bz2 was requested at [1]. A popular ISO compressed exclusively with bz2 is OPNsense [2]. Since this requires adding `bz2` to the list of known compression formats we add decompression methods for vmz a tar. [1] https://bugzilla.proxmox.com/show_bug.cgi?id=5267 [2] https://opnsense.org

[pve-devel] [PATCH manager] ui: backup job detail: fix wrong gettext

2024-08-06 Thread Dominik Csapak
'suspend' mode should not be shown as `gettext('Snapshot')` reported in the forum: https://forum.proxmox.com/threads/152365/ Signed-off-by: Dominik Csapak --- www/manager6/dc/BackupJobDetail.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/dc/BackupJobDetail.j