<font face="Default Sans Serif,Verdana,Arial,Helvetica,sans-serif" size="2"><font face="Verdana, Arial, Helvetica, sans-serif">My commit which adds the encryption engine configuration is default to "default".</font><div style="font-family: Verdana, Arial, Helvetica, sans-serif;">Only in the post-parse callback in the qemu driver, I switch it from "default" to "qemu".</div><div style="font-family: Verdana, Arial, Helvetica, sans-serif;">Thus, if you don't call the post-parse on this test, then tests which use luks format fail.</div><div style="font-family: Verdana, Arial, Helvetica, sans-serif;">Specifically:</div><div style="font-family: Verdana, Arial, Helvetica, sans-serif;"><br></div><div style=""><div style=""><font face="Verdana, Arial, Helvetica, sans-serif">network-qcow2-backing-chain-encryption_auth</font><br></div><div style=""><font face="Verdana, Arial, Helvetica, sans-serif">file-qcow2-backing-chain-encryption<br></font></div><div style=""><font face="Verdana, Arial, Helvetica, sans-serif">file-raw-luks<br></font></div><br><div style="font-family: Verdana, Arial, Helvetica, sans-serif;"><font color="#990099">-----"Peter Krempa" <<a href="mailto:pkrempa@redhat.com" target="_blank" rel="noopener noreferrer">pkrempa@redhat.com</a>> wrote: -----</font></div><div class="iNotesHistory" style="font-family: Verdana, Arial, Helvetica, sans-serif; padding-left: 5px;"><div style="padding-right:0px;padding-left:5px;border-left:solid black 2px;">To: "Or Ozeri" <<a href="mailto:oro@il.ibm.com" target="_blank" rel="noopener noreferrer">oro@il.ibm.com</a>><br>From: "Peter Krempa" <<a href="mailto:pkrempa@redhat.com" target="_blank" rel="noopener noreferrer">pkrempa@redhat.com</a>><br>Date: 10/06/2021 09:37AM<br>Cc: <a href="mailto:libvir-list@redhat.com" target="_blank" rel="noopener noreferrer">libvir-list@redhat.com</a>, <a href="mailto:idryomov@gmail.com" target="_blank" rel="noopener noreferrer">idryomov@gmail.com</a>, <a href="mailto:to.my.trociny@gmail.com" target="_blank" rel="noopener noreferrer">to.my.trociny@gmail.com</a>, <a href="mailto:dannyh@il.ibm.com" target="_blank" rel="noopener noreferrer">dannyh@il.ibm.com</a><br>Subject: [EXTERNAL] Re: [PATCH v2 1/5] qemu: add disk post parse to qemublocktest<br><br><div><font face="Courier New,Courier,monospace" size="2">On Tue, Oct 05, 2021 at 09:41:12 -0500, Or Ozeri wrote:<br>> The post parse callback is part of the real (non-test) processing flow.<br>> This commit adds it (for disks) to the qemublocktest flow as well.<br><br>Could you please elaborate why this is needed? Specifically<br>qemublocktest takes a few liberties from the "real" code flow since we<br>need to fake a lot of stuff. E.g. see testQemuDiskXMLToJSONFakeSecrets.<br><br>Specifically I didn't see anything in your patches [1] which would add<br>anything to the post parse callback.<br><br><br>[1] Well after my crude rebase of the series. What you've posted didn't<br>apply neither on master nor on the last release. I had to go one version<br>back and it had conflicts which I didn't spend much time thinking about.<br><br></font></div></div></div></div></font><BR>
<BR>