On Fri, Dec 16, 2016 at 10:07 AM, Guido Günther <[email protected]> wrote:

> The fix is to use qemu not kvm in the domain XML in this case.


If that is the way to go - to focus on testing qemu:/// - that is right.
The current call to virt-host-validate qemu already has a || true so it is
not fatal which is good.

What do you think of something like the following then (untested so far):
(this is based on my current d/t/control so your diff might be slightly
different.
That gets more similar again when I rebase to your latest code early next
week.)

diff --git a/debian/tests/control b/debian/tests/control
index c4f5244..cd15424 100644
--- a/debian/tests/control
+++ b/debian/tests/control
@@ -4,7 +4,7 @@ Restrictions: allow-stderr

Tests: smoke-qemu-session
Depends: libvirt-daemon-system, libvirt-clients, qemu-system,
libxml2-utils,
-         qemu-kvm
+         qemu-user
Restrictions: allow-stderr, isolation-machine, needs-root

Tests: smoke-lxc
diff --git a/debian/tests/smoke-qemu-session.xml
b/debian/tests/smoke-qemu-session.xml
index 47b33d5..c471ff2 100644
--- a/debian/tests/smoke-qemu-session.xml
+++ b/debian/tests/smoke-qemu-session.xml
@@ -1,4 +1,4 @@
-<domain type='kvm'>
+<domain type='qemu'>
  <name>sqs</name>
  <memory unit='KiB'>256000</memory>
  <currentMemory unit='KiB'>256000</currentMemory>
@@ -18,7 +18,7 @@
  <on_reboot>destroy</on_reboot>
  <on_crash>destroy</on_crash>
  <devices>
-    <emulator>/usr/bin/kvm</emulator>
+    <emulator>/usr/bin/qemu-x86_64</emulator>
    <controller type='virtio-serial' index='0'>
      <alias name='virtio-serial0'/>
      <address type='pci' domain='0x0000' bus='0x00' slot='0x04'
function='0x0'/>


-- 
Christian Ehrhardt
Software Engineer, Ubuntu Server
Canonical Ltd

Reply via email to