From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <pve-devel-bounces@lists.proxmox.com>
Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9])
	by lore.proxmox.com (Postfix) with ESMTPS id 647C51FF165
	for <inbox@lore.proxmox.com>; Thu,  5 Jun 2025 16:19:38 +0200 (CEST)
Received: from firstgate.proxmox.com (localhost [127.0.0.1])
	by firstgate.proxmox.com (Proxmox) with ESMTP id EAEF41A56D;
	Thu,  5 Jun 2025 16:19:57 +0200 (CEST)
Message-ID: <b48bed1d-96f7-41b2-bfb0-836da2306018@proxmox.com>
Date: Thu, 5 Jun 2025 16:19:55 +0200
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
To: "DERUMIER, Alexandre" <alexandre.derumier@groupe-cyllene.com>,
 "pve-devel@lists.proxmox.com" <pve-devel@lists.proxmox.com>
References: <mailman.197.1748937402.395.pve-devel@lists.proxmox.com>
 <59a59682-d54e-4ba2-94da-bbee253eeacc@proxmox.com>
 <0aa862ab988ff9df50f4637f3712f137f23c34c2.camel@groupe-cyllene.com>
Content-Language: en-US
From: Fiona Ebner <f.ebner@proxmox.com>
In-Reply-To: <0aa862ab988ff9df50f4637f3712f137f23c34c2.camel@groupe-cyllene.com>
X-SPAM-LEVEL: Spam detection results:  0
 AWL -0.032 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
 SPF_HELO_NONE           0.001 SPF: HELO does not publish an SPF Record
 SPF_PASS               -0.001 SPF: sender matches SPF record
Subject: Re: [pve-devel] [PATCH-SERIES v6 pve-storage/qemu-server/pve-qemu]
 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 <pve-devel.lists.proxmox.com>
List-Unsubscribe: <https://lists.proxmox.com/cgi-bin/mailman/options/pve-devel>, 
 <mailto:pve-devel-request@lists.proxmox.com?subject=unsubscribe>
List-Archive: <http://lists.proxmox.com/pipermail/pve-devel/>
List-Post: <mailto:pve-devel@lists.proxmox.com>
List-Help: <mailto:pve-devel-request@lists.proxmox.com?subject=help>
List-Subscribe: <https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel>, 
 <mailto:pve-devel-request@lists.proxmox.com?subject=subscribe>
Reply-To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: pve-devel-bounces@lists.proxmox.com
Sender: "pve-devel" <pve-devel-bounces@lists.proxmox.com>

Am 05.06.25 um 16:08 schrieb DERUMIER, Alexandre:
> Am 03.06.25 um 09:55 schrieb Alexandre Derumier via pve-devel:
>> This new patch series enable blockdev for qemu machine > 10.0 to
>> avoid breaking efidisk and maybe
>> potential migrations bug
> 
>>> Did you see any actual issues with migration or mirroring the EFI
>>> disk
>>> now or is this just a gut feeling?
> 
> I don't have tested it to be honest (because don't use efi in
> production, and I really don't known how to verify that it's working
> fine after a migration+ also the size param) , so it was more by
> security
> 
> but as it's attached to -machine... directly instead -drive, I really
> don't known.  and they was also the version gate for old machine
> version without raw support.
> 
> We also need to handle migration+storage motion for local dirve between
> pve8->pve9, and
> I don't have tested if pve8 can still use drive-mirror to pve9 nbd
> blockdev.
> 
> 
> I only have tested live vm migration with shared storage between pve8
> to pve9 without efi,
> and the switch from drive to blockdev was working fine.

I am inclined to go with version guarding too. I'm doubtful we can cover
all edge cases and all defaults even if we are very careful. There are
just too many possible options and combinations for drives. It's likely
already required for setting the 'write-cache' option on the front-end
drive device, since we didn't do that before. Maybe that won't change
the migration stream either, but it's better to be safe than sorry.


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