The idea of having a qemu/qemu-kvm variants block
turned out to be a bad idea, since we are never meant
to run them in combination. So let's roll back
this change and give some examples on tests.cfg.

Also, cleaned up tests.cfg.sample, and introduced
Fedora 12 as the default host, as it is the latest
Fedora upstream.

Signed-off-by: Lucas Meneghel Rodrigues <l...@redhat.com>
---
 client/tests/kvm/tests.cfg.sample      |   48 ++++++--------------------------
 client/tests/kvm/tests_base.cfg.sample |    8 -----
 2 files changed, 9 insertions(+), 47 deletions(-)

diff --git a/client/tests/kvm/tests.cfg.sample 
b/client/tests/kvm/tests.cfg.sample
index bde4efa..c478ba2 100644
--- a/client/tests/kvm/tests.cfg.sample
+++ b/client/tests/kvm/tests.cfg.sample
@@ -20,66 +20,36 @@ include cdkeys.cfg
 #cdrom.* ?<= /tmp/kvm_autotest_root/
 #steps ?<= /tmp/kvm_autotest_root/
 
-# You will notice that in all test definition blocks we have 'only qemu-kvm'
-# set. This means that qemu-kvm command line syntax will be used. If you
-# intend to test qemu upstream, you'll have to change that to 'only qemu'.
 variants:
     - @full:
-        only qemu-kvm
-    - @sample1:
-        only qemu-kvm
-        only no_pci_assignable
-        only Fedora Windows
-    - @sample2:
+    - @fc12_qemu:
         only qemu-kvm
         only qcow2
         only ide
         only default
         only up
         only no_pci_assignable
-        only Fedora.9.* RHEL.5.* Windows
-        only rtl8139
-    - @sample3:
-        only qemu-kvm
-        only no_pci_assignable
-        only qcow2.*ide.*default.*up.*Ubuntu-8.10-server.*(autotest.sleeptest)
-        only rtl8139
-    - @fc8_step:
-        only qemu-kvm
-        only qcow2
-        only ide
-        only default
-        only up
-        only no_pci_assignable
-        only Fedora.8.32
-        only install setup boot shutdown
-        only rtl8139
-        only hugepages
-    - @winXP_32_unattended:
-        only qemu-kvm
-        only qcow2
-        only ide
-        only default
-        only up
-        only no_pci_assignable
-        only WinXP.32
+        only Fedora.12.64
         no install setup
         no hugepages
         only unattended_install boot shutdown
         only rtl8139
-    - @fc11_kickstart:
+        # The only change needed to test qemu upstream
+        # instead of qemu-kvm is the explicit -enable-kvm
+        # that has to be passed to the command line
+        extra_params = ' -enable-kvm'
+    - @fc12_qemu_kvm:
         only qemu-kvm
         only qcow2
         only ide
         only default
         only up
         only no_pci_assignable
-        only Fedora.11.64
+        only Fedora.12.64
         no install setup
         no hugepages
         only unattended_install boot shutdown
         only rtl8139
 
 # Choose your test list from the testsets defined
-only fc11_kickstart
-
+only fc12_qemu_kvm
diff --git a/client/tests/kvm/tests_base.cfg.sample 
b/client/tests/kvm/tests_base.cfg.sample
index 2d03f1f..8a7a13d 100644
--- a/client/tests/kvm/tests_base.cfg.sample
+++ b/client/tests/kvm/tests_base.cfg.sample
@@ -62,8 +62,6 @@ variants:
 
     - unattended_install:
         type = unattended_install
-        # Unattended install also perform guest setup, so setup is not needed
-        no setup
         kill_vm_on_error = yes
         force_create_image = yes
         pre_command = scripts/unattended.py
@@ -894,12 +892,6 @@ linux_s3:
 
 
 variants:
-    - @qemu-kvm:
-    - qemu:
-        extra_params += " -enable-kvm"
-
-
-variants:
     - @up:
         no autotest.npb
     - smp2:
-- 
1.6.6

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to