<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 25, 2020 at 12:44 AM Peter Krempa <<a href="mailto:pkrempa@redhat.com">pkrempa@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">On Tue, Nov 24, 2020 at 15:25:01 -0600, Dustan B Helm wrote:<br>
> On Tue, Nov 24, 2020 at 12:59 AM Peter Krempa <<a href="mailto:pkrempa@redhat.com" target="_blank">pkrempa@redhat.com</a>> wrote:<br>
> > On Mon, Nov 23, 2020 at 17:17:15 -0600, Dustan B Helm wrote:<br>
<br>
[...]<br><br>
- any XML addition requires tests to be added<br>
<br>
    In this case you'll be also needing to add a qemu commandline test<br>
    later so I suggest you add your XML test as a new file to:<br>
    tests/qemuxml2argvdata, and enable the test in<br>
    tests/qemuxml2xmltest.c (output file is stored in<br>
    tests/qemuxml2xmloutdata, you can also run the test with<br>
    VIR_TEST_REGENREATE_OUTPUT=1 env variable set which generates the<br>
    proper output file)<br>
<br>
    Always use one of the _CAPS_LATEST macros for invoking the test<br>
    regardless of prior art.<br>
<br>
    You can add multiple disks to the tested XML to test any combination<br>
    of the <nfs> parameters.<br>
<br>
    Please make sure to list at least one example of the NFS disk being<br>
    a <backingStore> of a <source><br>
    (see tests/qemuxml2argvdata/disk-backing-chains.xml for inspiration)<br>
<br>
    Don't forget to also document the new elements in<br>
    docs/formatdomain.rst<br></blockquote><div><span style="background-color:transparent;color:rgb(0,0,0);font-family:Arial;white-space:pre-wrap">We don’t understand exactly how the testing frameworks are functioning. With regards to our XML testing, the tests/qemuxml2argvdata/ directory contains for each test both an xml file and an args file- is this .args file the expected output qemu command we want to generate from the input .xml file?</span></div><span id="gmail-docs-internal-guid-aec0ed62-7fff-def5-ed71-e92bfb4ea4b1"><p dir="ltr" style="line-height:1.8;margin-top:12pt;margin-bottom:12pt"><span style="font-family:Arial;color:rgb(0,0,0);background-color:transparent;font-variant-numeric:normal;font-variant-east-asian:normal;vertical-align:baseline;white-space:pre-wrap">If so, how do we invoke the VIR_TEST_REGENERATE_OUTPUT flag to create this .args file, and how concerned should we be about non-disk information provided in the .xml file as input? Additionally, since our source name (a file path) needs to include the NFS export at the beginning, how would we set up the XML input? Can the file path and NFS export be chosen arbitrarily? </span></p></span><br class="gmail-Apple-interchange-newline"></div></div>