From: Fiona Ebner <f.ebner@proxmox.com>
To: "DERUMIER, Alexandre" <alexandre.derumier@groupe-cyllene.com>,
"pve-devel@lists.proxmox.com" <pve-devel@lists.proxmox.com>,
"f.gruenbichler@proxmox.com" <f.gruenbichler@proxmox.com>
Subject: Re: [pve-devel] [PATCH-SERIES v3 pve-storage/qemu-server/pve-qemu] add external qcow2 snapshot support
Date: Mon, 13 Jan 2025 14:42:31 +0100 [thread overview]
Message-ID: <1fe23bd4-6bac-486e-9dc3-9d85386c034f@proxmox.com> (raw)
In-Reply-To: <2cbef7d2a33ed5ea6fab15b97f611fc4bf207c0f.camel@groupe-cyllene.com>
Am 13.01.25 um 12:58 schrieb DERUMIER, Alexandre:
>
>
>>> For almost all QMP commands, we only need to care about the node
>>> that's
>>> inserted for the drive.
> (yes, that the throttle group in my implementation, and I have a fixed
> name, I'm reusing the "drive-(ide|scsi|virtio)x naming"
>
This assumption might not always hold, see my earlier replies. To avoid
relying on the assumption, we can use query-block and warn if the name
doesn't match the fixed one we expect. This usually shouldn't happen,
but who knows what the future brings and what else touches the block
graph. Querying is cheap and will immediately give us a good idea
what/when something goes wrong. Such a situation might even just be a
valid edge case or third-party use case. I don't want to break those for
no reason.
Let me illustrate what I mean with an example: say that a user requests
move storage for drive scsi0. Then we can:
1. use query-block to get the node name of what's inserted in drive
scsi0 right now
2. warn if the node-name doesn't match the expected drive-scsi0 name
3. run blockdev-mirror with the node name we queried, because that is
the node name with that data that the guest also sees right now
We could also error out instead of warn in step 2, but that might break
some third-party use cases or edge cases we are not aware of right now.
The problem for how to name the nodes in the backing chain is
independent from this. I'll answer the other mail where you discussed
this with Fabian.
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
prev parent reply other threads:[~2025-01-13 13:43 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-16 9:12 Alexandre Derumier via pve-devel
2025-01-09 14:13 ` Fabian Grünbichler
2025-01-10 7:44 ` DERUMIER, Alexandre via pve-devel
2025-01-10 9:55 ` Fiona Ebner
2025-01-10 12:30 ` DERUMIER, Alexandre via pve-devel
[not found] ` <8f309dfe189379acf72db07398a37a98e8fc3550.camel@groupe-cyllene.com>
2025-01-13 10:06 ` Fiona Ebner
2025-01-13 10:54 ` Fiona Ebner
2025-01-13 10:57 ` DERUMIER, Alexandre via pve-devel
2025-01-13 11:54 ` Fiona Ebner
2025-01-13 11:58 ` DERUMIER, Alexandre via pve-devel
[not found] ` <2cbef7d2a33ed5ea6fab15b97f611fc4bf207c0f.camel@groupe-cyllene.com>
2025-01-13 13:42 ` Fiona Ebner [this message]
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=1fe23bd4-6bac-486e-9dc3-9d85386c034f@proxmox.com \
--to=f.ebner@proxmox.com \
--cc=alexandre.derumier@groupe-cyllene.com \
--cc=f.gruenbichler@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