Re: [PATCH] qemuDomainChangeNet: Error when boot index changes in live XML
On a Friday in 2024, Adam Julis wrote:
If the original code detected a missing or null boot index in the
new XML, it automatically added the current value. This
autocompletion was incorrect because it was impossible to
distinguish between user intent and user error - changing the
This is also true for other attributes qemuDomainChangeNet copies over
from the old definition.
If you're feeling brave, you can clean those up too.
I'm not sure what the risks are of someone relying on net-update working
with an incomplete XML definition are, but:
1) we amended the API documentation to say it needs the full device XML:
commit 2864b4cd1ccda9c12f0c0ceb340836c42a2a0e52
CommitDate: 2018-09-04 10:28:40 +0200
virDomainDetachDeviceFlags: Clarify update semantics
2) we did not extend the "autocompletion" to other attributes, so over
time the likelihood of net-update doing the right thing has decreased
Jano
boot order itself is forbidden and should always be an error.
Resolves: https://issues.redhat.com/browse/RHEL-23416
Fixies: Commit hash aa3e07caec6179dfa6479deab14a21a493637d53
Signed-off-by: Adam Julis <ajulis(a)redhat.com>
---
src/qemu/qemu_hotplug.c | 2 --
1 file changed, 2 deletions(-)
Attachments:
- signature.asc (application/pgp-signature — 228 bytes)