<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 21, 2021 at 10:54 PM Michal Prívozník <<a href="mailto:mprivozn@redhat.com">mprivozn@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
> The bug quoted shows a few examples where we fail to save status.<br>
> <br>
> I'm very surprised we don't save status when hotplugging a NIC or a<br>
> disk, as the BZ suggests.<br>
<br>
I'm not convinced that the steps there are 100% correct. We do call<br>
virDomainObjSave() after live attach:<br>
<br>
<a href="https://gitlab.com/libvirt/libvirt/-/blob/master/src/qemu/qemu_driver.c#L7834" rel="noreferrer" target="_blank">https://gitlab.com/libvirt/libvirt/-/blob/master/src/qemu/qemu_driver.c#L7834</a><br>
<br></blockquote><div>The two steps for hot-unplug in the BZ actually failed.</div><div>(sorry I didn't mention the result in the steps)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
> Missing status save in QMP monitor command passthrough is less<br>
> surprising though since we're not actually changing the VM state<br>
> when doing that, so would not have reason to save state except<br>
> for the taint message.<br>
<br>
Yep. For a few cases it is hidden in BeginJob() and EndJob() but not for<br>
agent jobs.<br>
<br>
Michal<br>
<br>
</blockquote></div></div>