diff mbox

virt.virt_utils: Fix hardcoded test URL in run_tests()

Message ID 1304353595-8951-1-git-send-email-lmr@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Lucas Meneghel Rodrigues May 2, 2011, 4:26 p.m. UTC 
Now that we aim to have different implementations of
virtualization tests, the utility function run_tests()
can't have the 'kvm' url hardcoded in the function. Rather,
it needs to pick up 'vm_type' from the test params.

Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
---
 client/virt/virt_utils.py |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)
diff mbox

Patch

diff --git a/client/virt/virt_utils.py b/client/virt/virt_utils.py
index 398ecd4..5510c89 100644
--- a/client/virt/virt_utils.py
+++ b/client/virt/virt_utils.py
@@ -1170,7 +1170,8 @@  def run_tests(parser, job):
             # We need only one execution, profiled, hence we're passing
             # the profile_only parameter to job.run_test().
             profile_only = bool(profilers) or None
-            current_status = job.run_test_detail("kvm", params=dict,
+            current_status = job.run_test_detail(dict.get("vm_type"),
+                                                 params=dict,
                                                  tag=test_tag,
                                                  iterations=test_iterations,
                                                  profile_only=profile_only)
@@ -1179,7 +1180,8 @@  def run_tests(parser, job):
         else:
             # We will force the test to fail as TestNA during preprocessing
             dict['dependency_failed'] = 'yes'
-            current_status = job.run_test_detail("kvm", params=dict,
+            current_status = job.run_test_detail(dict.get("vm_type"),
+                                                 params=dict,
                                                  tag=test_tag,
                                                  iterations=test_iterations)