Re: [pve-devel] [PATCH pve-storage 08/10] qcow2: add external snapshot support

From: "Fabian Grünbichler" <f.gruenbichler@proxmox.com>
To: "DERUMIER, Alexandre" <alexandre.derumier@groupe-cyllene.com>,
	"pve-devel@lists.proxmox.com" <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] [PATCH pve-storage 08/10] qcow2: add external snapshot support
Date: Mon, 7 Jul 2025 12:53:16 +0200 (CEST)	[thread overview]
Message-ID: <1109363863.2769.1751885596032@webmail.proxmox.com> (raw)
In-Reply-To: <c671fe82a7cdab90a3691115a7132d0a35ae79b7.camel@groupe-cyllene.com>

> DERUMIER, Alexandre <alexandre.derumier@groupe-cyllene.com> hat am 07.07.2025 12:18 CEST geschrieben:

> >>for the DirPlugin we need to add another level for the snapshots
> >>on-disk, else we cannot differentiate between weirdly-named image
> >>files and snapshot files..
> >>
> >>so we could add a directory "snapshots" and put all the snapshot
> >>files in there. since snapshots are never references a volume ID,
> >>this just affects the on disk structure, not the volume ID format.
> 
> ok, no problem, I'll do it like this.
> 
> Do you want the a snapshot sub directory like ?
> /var/lib/vz/images/<vmid>/snapshots/ 

I think this makes the most sense, as it logically groups the snapshot
files below the main ones..

> 
> or a global snapshots like
> 
> /var/lib/vz/snapshots/<vmid>/
> 
> ?

_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel