From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9]) by lore.proxmox.com (Postfix) with ESMTPS id 3F4381FF164 for ; Fri, 4 Jul 2025 13:58:03 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 7A24637007; Fri, 4 Jul 2025 13:58:42 +0200 (CEST) Date: Fri, 4 Jul 2025 13:58:38 +0200 (CEST) From: =?UTF-8?Q?Fabian_Gr=C3=BCnbichler?= To: Proxmox VE development discussion Message-ID: <1861462900.2272.1751630318843@webmail.proxmox.com> In-Reply-To: References: MIME-Version: 1.0 X-Priority: 3 Importance: Normal X-Mailer: Open-Xchange Mailer v7.10.6-Rev79 X-Originating-Client: open-xchange-appsuite X-SPAM-LEVEL: Spam detection results: 0 AWL -0.065 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment RCVD_IN_VALIDITY_CERTIFIED_BLOCKED 0.218 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. RCVD_IN_VALIDITY_RPBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. RCVD_IN_VALIDITY_SAFE_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record URIBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to URIBL was blocked. See http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more information. [plugin.pm, lvmthinplugin.pm, rbdplugin.pm, dirplugin.pm, blockdev.pm, storage.pm, iscsidirectplugin.pm, pbsplugin.pm, qemuconfig.pm, qemuserver.pm, snapshot-test.pm, lvmplugin.pm, qemuimage.pm, esxiplugin.pm, zfspoolplugin.pm, common.pm] Subject: Re: [pve-devel] [PATCH-SERIES v7 pve-storage/qemu-server] add external qcow2 snapshot support X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Proxmox VE development discussion Cc: Thomas Lamprecht Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: pve-devel-bounces@lists.proxmox.com Sender: "pve-devel" > Alexandre Derumier via pve-devel hat am 04.07.2025 08:44 CEST geschrieben: > This patch series implement qcow2 external snapshot support for files && lvm volumes > > The current internal qcow2 snapshots have bad write performance because no metadatas can be preallocated. > > This is particulary visible on a shared filesystem like ocfs2 or gfs2. > > Also other bugs are freeze/lock reported by users since years on snapshots delete on nfs > (The disk access seem to be frozen during all the delete duration) > > This also open doors for remote snapshot export-import for storage replication. sent some feedback, mostly for the storage and qemu-server <-> storage interface parts, haven't yet done an in-depth pass over qemu-server or end-to-end. in particular, the rename_volume vs rename_snapshot and the Plugin snapshot file name/.. questions should probably be solved before the next version it would also be great if you could add a patch bumping API version and age and documenting the changes/new parameters/interfaces - also so that we can ensure we are on the same page and haven't overlooked anything. AFAICT it should all be new opt-in or backwards compatible changes, once the feedback is incorporated. > > Changelog v7: > - rebased on fiona blockdev work > > MAIN TODO: > - add snapshots tests in both pve-storage && qemu-server > - better handle snapshot failure with multiple disks > > pve-storage: > > Alexandre Derumier (10): > tests: add lvmplugin test > common: add qemu_img_create an preallocation_cmd_option > common: qemu_img_create: add backing_file support > rename_volume: add source && target snap > common: add qemu_img_info helper > common: add qemu-img measure > storage: volume_snapshot: add $running param > qcow2: add external snapshot support > lvmplugin: add qcow2 snapshot > storage : add volume_support_qemu_snapshot > > src/PVE/Storage.pm | 34 +- > src/PVE/Storage/Common.pm | 116 ++++++ > src/PVE/Storage/DirPlugin.pm | 11 + > src/PVE/Storage/ESXiPlugin.pm | 2 +- > src/PVE/Storage/ISCSIDirectPlugin.pm | 2 +- > src/PVE/Storage/LVMPlugin.pm | 519 ++++++++++++++++++++---- > src/PVE/Storage/LvmThinPlugin.pm | 2 +- > src/PVE/Storage/PBSPlugin.pm | 2 +- > src/PVE/Storage/Plugin.pm | 376 ++++++++++++----- > src/PVE/Storage/RBDPlugin.pm | 8 +- > src/PVE/Storage/ZFSPoolPlugin.pm | 2 +- > src/test/Makefile | 5 +- > src/test/run_test_lvmplugin.pl | 577 +++++++++++++++++++++++++++ > 13 files changed, 1465 insertions(+), 191 deletions(-) > create mode 100755 src/test/run_test_lvmplugin.pl > > qemu-server: > > Alexandre Derumier (3): > qemu_img convert : add external snapshot support > blockdev: add backing_chain support > qcow2: add external snapshot support > > src/PVE/QemuConfig.pm | 4 +- > src/PVE/QemuServer.pm | 132 +++++-- > src/PVE/QemuServer/Blockdev.pm | 345 +++++++++++++++++- > src/PVE/QemuServer/QemuImage.pm | 6 +- > src/test/cfg2cmd/simple-backingchain.conf | 25 ++ > src/test/cfg2cmd/simple-backingchain.conf.cmd | 33 ++ > src/test/run_config2command_tests.pl | 47 +++ > src/test/run_qemu_img_convert_tests.pl | 59 +++ > src/test/snapshot-test.pm | 4 +- > 9 files changed, 620 insertions(+), 35 deletions(-) > create mode 100644 src/test/cfg2cmd/simple-backingchain.conf > create mode 100644 src/test/cfg2cmd/simple-backingchain.conf.cmd > > -- > 2.39.5 _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel