Re: [pve-devel] [RFC manager 1/2] fix #5244 pveceph: install: add new repository for offline installation

2025-05-28 Thread Aaron Lauterer
Sent a v1: https://lore.proxmox.com/pve-devel/20250528164630.2204379-1-a.laute...@proxmox.com/T/#t ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH manager v1 3/3] pveceph: print repo metadata when installing from manual

2025-05-28 Thread Aaron Lauterer
By printing the site and component, the person installing it manuall has a final check to see if the correct repository is being used. Signed-off-by: Aaron Lauterer --- Notes: I intially tried to get the repo definition similar to how it is in the .list file and printed by apt-cache poli

[pve-devel] [PATCH manager v1 1/3] fix #5244 pveceph: install: add new repository for offline installation

2025-05-28 Thread Aaron Lauterer
by adding a 4th repository option called 'offline'. If set, the ceph installation step will not touch the repository configuration. We add a simple version check to make sure that the latest version available (and to be installed) does match the selected major Ceph version. Signed-off-by: Aaron L

[pve-devel] [PATCH manager v1 2/3] ui: CephInstallWizard: add option and hint for offline repository

2025-05-28 Thread Aaron Lauterer
The new 'offline' repository option will not try to configure the Ceph repositories during installation. Signed-off-by: Aaron Lauterer Tested-by: Christoph Heiss --- Notes: changes since RFC: * renamed option from 'offline' to 'manual' and therefore adapted the dropdown name

[pve-devel] Cluster storage plugin assistant service

2025-05-28 Thread Andrei Perapiolkin via pve-devel
--- Begin Message --- Hi Proxmox Community, I am working on implementing custom storage plugin and after evaluating various edge cases related to volume live migration, I have decided to create a 'storage assistant service'. This service will be responsible for managing my storage plugin volume

Re: [pve-devel] Volume live migration concurrency

2025-05-28 Thread Andrei Perapiolkin via pve-devel
--- Begin Message --- Hi Fabian, Thank you for your time dedicated to this issue. My current understanding is that all assets related to snapshots should to be removed when volume is deactivation, is it correct? Or all volumes and snapshots expected to be present across the entire cluster unti

Re: [pve-devel] [PATCH qemu-server 13/14] blockdev: add backing_chain support

2025-05-28 Thread DERUMIER, Alexandre via pve-devel
--- Begin Message --- Message initial De: Fiona Ebner À: Proxmox VE development discussion , f.gruenbich...@proxmox.com Cc: "DERUMIER, Alexandre" Objet: Re: [pve-devel] [PATCH qemu-server 13/14] blockdev: add backing_chain support Date: 28/05/2025 12:07:14 Am 28.05.25 um 11:08

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

2025-05-28 Thread Stefan Hanreich
Tested the following things and they worked: * Create a new simple zone with Nautobot IPAM * Create a VNet with a Subnet + GW -> IP allocated * Create a VM on that VNet -> IP allocated * Create a CT on that VNet -> IP allocated * Change network device to another bridge -> IP deleted * Change back

[pve-devel] [PATCH pve_flutter_frontend] ui: Changed "sites" to "hosts".

2025-05-28 Thread Alexander Abraham
The word "sites" is a little bit inaccurate in this instance. "Hosts", on the other hnad more accurately describes any PVE machines users may have. Signed-off-by: Alexander Abraham --- lib/pages/main_layout_slim.dart | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/pages/m

[pve-devel] [PATCH proxmox_login_manager] ui: Changed "sites" to "hosts".

2025-05-28 Thread Alexander Abraham
Changed the "Available Sites" text inside the added PVE hosts view to "Available Hosts" because "Sites" is a bit odd. Signed-off-by: Alexander Abraham --- lib/proxmox_login_selector.dart | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/proxmox_login_selector.dart b/lib/pro

[pve-devel] applied: [PATCH qemu-server] tests: qemu_img_convert: add rbd and btrfs snapshots convert test

2025-05-28 Thread Fiona Ebner
Applied, thanks! I changed the path/pool to not re-use existing ones for different storages, just for future-proofing/more realism. [1/1] tests: qemu_img_convert: add rbd and btrfs snapshots convert test commit: 1fd1ca60f94c68702a6ebda1000e0e9d5fc7f50f _

Re: [pve-devel] [PATCH ifupdown2 0/7] bump to upstream 3.9.0 release

2025-05-28 Thread Christoph Heiss
Ping, still applies. On Wed Apr 23, 2025 at 12:45 PM CEST, Christoph Heiss wrote: > Going by the changelog, there seem to be no actual breaking changes, > except one commit - which is reverted w/ patch #4, due to being broken. > > Other than that, it's just bug fixes and new attributes. I also ski

Re: [pve-devel] [PATCH qemu-server 13/14] blockdev: add backing_chain support

2025-05-28 Thread Fiona Ebner
Am 28.05.25 um 11:08 schrieb DERUMIER, Alexandre via pve-devel: > perl question: how to call a resursive private sub ? (it don't seem to > allow it) AFAIK, you can do it by declaring it up-front: > [I] root@pve8a1 ~# cat asdf.pm > use strict; > use warnings; > > package Private; > > my sub add

Re: [pve-devel] [PATCH pve-docs v6] fix #5644: Moved the swap partition section

2025-05-28 Thread Fiona Ebner
Please use present tense in the commit title/subject like is customary in Proxmox VE development. Am 27.05.25 um 14:39 schrieb Alexander Abraham: > The section about "Creating SWAP partitions" lives in its own > file and the warning about creating SWAP partitions on ZFS was > moved back to the ori

[pve-devel] [PATCH manager 4/4] ui: dc: tag edit: use more specific documentation section

2025-05-28 Thread Christoph Heiss
Now that there is a dedicated section for configuring tags, link to that instead of to the documentation of the datacenter configuration file. Signed-off-by: Christoph Heiss --- www/manager6/dc/OptionView.js | 2 +- www/manager6/dc/RegisteredTagsEdit.js | 2 +- www/manager6/dc/UserTagAcc

[pve-devel] [PATCH manager 3/4] ui: panel: TagConfig: link to more specific online help section

2025-05-28 Thread Christoph Heiss
This section is available since pve-docs 8.2.4, released November 2024. Includes the necessary bump to accommodate that. Signed-off-by: Christoph Heiss --- Hopefully the bump is okay as-is, but feel free to adjust as needed. debian/control | 2 +- www/manager6/panel/TagConfig.j

[pve-devel] [PATCH manager 1/4] ui: qemu: AudioEdit: enable online help link

2025-05-28 Thread Christoph Heiss
This section has been available since pve-docs 6.0-5, which was released in October 2019. Signed-off-by: Christoph Heiss --- www/manager6/qemu/AudioEdit.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/www/manager6/qemu/AudioEdit.js b/www/manager6/qemu/AudioEdit.js index

[pve-devel] [PATCH manager 0/4] improve some onlineHelp links

2025-05-28 Thread Christoph Heiss
Came across these commented-out `onlineHelp` and looked into it - all referenced sections across the repo have been (very) long available in pve-docs and can thus be enabled. Additionally, after looking at #3, thought to link to this "new" section in some other tags-related panels, instead of the

[pve-devel] [PATCH manager 2/4] ui: qemu: HDTPM: enable online help link

2025-05-28 Thread Christoph Heiss
This section is available since pve-docs 7.1-1, which was released in November 2021. Signed-off-by: Christoph Heiss --- www/manager6/qemu/HDTPM.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/qemu/HDTPM.js b/www/manager6/qemu/HDTPM.js index 87349aedb..dac3bb5e

Re: [pve-devel] [PATCH qemu-server 13/14] blockdev: add backing_chain support

2025-05-28 Thread DERUMIER, Alexandre via pve-devel
--- Begin Message --- > >   > +sub generate_backing_blockdev { >>make this private? perl question: how to call a resursive private sub ? (it don't seem to allow it) > +    my ($storecfg, $snapshots, $deviceid, $drive, $snap_id) = @_; > + > +    my $snapshot = $snapshots->{$snap_id}; > +    my

Re: [pve-devel] [PATCH network] sdn: factor out frr config generation and writing

2025-05-28 Thread Gabriel Goller
Obsolete due to: https://lore.proxmox.com/pve-devel/20250522161731.537011-1-s.hanre...@proxmox.com/ ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH qemu-server] qm: import: use schema variant with 'import-from' disk allocation support

2025-05-28 Thread Fiona Ebner
As reported in the community forum [0], it was not possible to use the extended schema that supports disk allocation with 'import-from' for 'qm import'. Align it more closely to the VM creation endpoint. [0]: https://forum.proxmox.com/threads/166734/ Signed-off-by: Fiona Ebner --- Better viewed

Re: [pve-devel] Volume live migration concurrency

2025-05-28 Thread Fabian Grünbichler
> Andrei Perapiolkin hat am 27.05.2025 18:08 > CEST geschrieben: > > > > 3. In the context of live migration: Will Proxmox skip calling > > /deactivate_volume/ for snapshots that have already been activated? > > Should the storage plugin explicitly deactivate all snapshots of a > > volume dur