From: Fiona Ebner <f.ebner@proxmox.com>
To: Daniel Kral <d.kral@proxmox.com>,
Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] [PATCH qemu-server] migration: prohibit renaming cloud-init drive
Date: Tue, 10 Mar 2026 13:38:05 +0100 [thread overview]
Message-ID: <0f6d46a7-ee96-47c6-8c60-ce7348166d4a@proxmox.com> (raw)
In-Reply-To: <4f619b8f-b0d3-4163-af25-e53272aa7bea@proxmox.com>
Am 10.03.26 um 1:30 PM schrieb Fiona Ebner:
> Am 10.03.26 um 1:22 PM schrieb Daniel Kral:
>> On Mon Dec 1, 2025 at 2:51 PM CET, Fiona Ebner wrote:
>>> Usually, disks are allowed to be renamed during migration if there is
>>> a naming conflict caused by a left-over disk on the target. However,
>>> the type of the cloud-init disk is encoded in its name, so it must not
>>> be renamed or it cannot be recognized as a cloud-init disk anymore, as
>>> reported in the community forum [0].
>>>
>>> [0]: https://forum.proxmox.com/threads/167767/
>>
>> A user in the community forum [1] reported that if a VM is moved to
>> another node during a fence recovery, the VM will fail to migrate back,
>> e.g., if a node affinity rule prioritizes the previous node.
>>
>> This patch is correct to not allow renaming cloudinit images, but I
>> wonder if it would be a reasonable idea to allow overriding existing
>> cloudinit images on the target node as these are auto-generated?
>>
>> That would probably need a `--allow-override` flag for pvesm, which
>> would require that the target host can understand the parameter though.
>>
>> [1] https://forum.proxmox.com/threads/181516/
>
> I think we could remove left-over cloud-init images as part of the
> 'pvesr prepare-local-job' on the target?
Ah wait, it doesn't only affect the scenario with replication. The
'pvesm apiinfo' command was added when introducing the '--allow-rename'
flag. You could add '--allow-override', bump the API and then use that
call to check if it is understood by the target.
Or maybe we can add some kind of preparatory tunnel command for removing
any left-over cloud-init images?
next prev parent reply other threads:[~2026-03-10 12:38 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-12-01 13:51 Fiona Ebner
2026-01-20 17:21 ` [pve-devel] applied: " Thomas Lamprecht
2026-03-10 12:22 ` [pve-devel] " Daniel Kral
2026-03-10 12:31 ` Fiona Ebner
2026-03-10 12:38 ` Fiona Ebner [this message]
2026-03-10 14:11 ` Daniel Kral
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=0f6d46a7-ee96-47c6-8c60-ce7348166d4a@proxmox.com \
--to=f.ebner@proxmox.com \
--cc=d.kral@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox