Re: [PATCH V2 2/8] tests: move sexpr2xml tests to WITH_LIBXL

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/09/2018 08:32 AM, Daniel P. Berrangé wrote:
On Fri, Apr 06, 2018 at 02:44:54PM -0600, Jim Fehlig wrote:
In preparation of removing the legacy Xen driver, move the
sexpr2xml tests from WITH_XEN to WITH_LIBXL. Even though the
legacy driver will be removed, we'll want to maintain the ability
to convert sexpr to XML. Requires fixing up the tests to account
for different behavior of Xen vs libxl post parse functions.

There is some test file fallout due to differences in handling
of default values between xend and libxl.


diff --git a/tests/sexpr2xmldata/sexpr2xml-boot-grub.xml b/tests/sexpr2xmldata/sexpr2xml-boot-grub.xml
index b9a8716b2..4b9f535fc 100644
--- a/tests/sexpr2xmldata/sexpr2xml-boot-grub.xml
+++ b/tests/sexpr2xmldata/sexpr2xml-boot-grub.xml
@@ -15,7 +15,7 @@
    <on_crash>destroy</on_crash>
    <devices>
      <disk type='block' device='disk'>
-      <driver name='phy'/>
+      <driver name='phy' type='raw'/>
        <source dev='/dev/MainVG/GuestVG'/>
        <target dev='xvda' bus='xen'/>
      </disk>
@@ -24,6 +24,5 @@
      </console>
      <input type='mouse' bus='xen'/>
      <input type='keyboard' bus='xen'/>
-    <memballoon model='xen'/>

For this I wonder if a better solution is actually to make the libxl
driver fill in the memballoon device. IIUC, you can't actually turn
it off, so all libxl guests will have this regardless.

Yes, good point.

So we should
just need to add the element a post-parse callback, and check for
unsupported "model" value when starting the guest

I'll send a follow up along these lines. In the meantime I've pushed this series. Good riddance xend...

Regards,
Jim

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux