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 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.  So we should
just need to add the element a post-parse callback, and check for
unsupported "model" value when starting the guest


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
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