diff mbox

[KVM-AUTOTEST,04/17] Modify run_autotest() in kvm_tests.py to use the new kvm_subprocess module.

Message ID 53d7f316f181c7efb429e7b9f138ed3e8b239cce.1248102188.git.mgoldish@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Michael Goldish July 20, 2009, 3:07 p.m. UTC
Signed-off-by: Michael Goldish <mgoldish@redhat.com>
---
 client/tests/kvm/kvm_tests.py |    9 +++++----
 1 files changed, 5 insertions(+), 4 deletions(-)

Comments

Lucas Meneghel Rodrigues July 23, 2009, 4:03 a.m. UTC | #1
Applied.

On Mon, Jul 20, 2009 at 12:07 PM, Michael Goldish<mgoldish@redhat.com> wrote:
> Signed-off-by: Michael Goldish <mgoldish@redhat.com>
> ---
>  client/tests/kvm/kvm_tests.py |    9 +++++----
>  1 files changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/client/tests/kvm/kvm_tests.py b/client/tests/kvm/kvm_tests.py
> index 2d11fed..5991aed 100644
> --- a/client/tests/kvm/kvm_tests.py
> +++ b/client/tests/kvm/kvm_tests.py
> @@ -1,6 +1,6 @@
>  import time, os, logging
>  from autotest_lib.client.common_lib import utils, error
> -import kvm_utils, ppm_utils, scan_results
> +import kvm_utils, kvm_subprocess, ppm_utils, scan_results
>
>  """
>  KVM test definitions.
> @@ -274,15 +274,16 @@ def run_autotest(test, params, env):
>     cmd += " --exclude=*.pyc"
>     cmd += " --exclude=*.svn"
>     cmd += " --exclude=*.git"
> -    kvm_utils.run_bg(cmd % (test.bindir, tarred_autotest_path), timeout=30)
> +    kvm_subprocess.run_fg(cmd % (test.bindir, tarred_autotest_path), timeout=30)
>
>     # tar the contents of bindir/autotest/tests/<test_name>
>     cmd = "cd %s; tar cvjf %s %s/*"
>     cmd += " --exclude=*.pyc"
>     cmd += " --exclude=*.svn"
>     cmd += " --exclude=*.git"
> -    kvm_utils.run_bg(cmd % (os.path.join(test.bindir, "autotest", "tests"),
> -                            tarred_test_path, test_name), timeout=30)
> +    kvm_subprocess.run_fg(cmd %
> +                          (os.path.join(test.bindir, "autotest", "tests"),
> +                           tarred_test_path, test_name), timeout=30)
>
>     # Check if we need to copy autotest.tar.bz2
>     copy = False
> --
> 1.5.4.1
>
> _______________________________________________
> Autotest mailing list
> Autotest@test.kernel.org
> http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
>
diff mbox

Patch

diff --git a/client/tests/kvm/kvm_tests.py b/client/tests/kvm/kvm_tests.py
index 2d11fed..5991aed 100644
--- a/client/tests/kvm/kvm_tests.py
+++ b/client/tests/kvm/kvm_tests.py
@@ -1,6 +1,6 @@ 
 import time, os, logging
 from autotest_lib.client.common_lib import utils, error
-import kvm_utils, ppm_utils, scan_results
+import kvm_utils, kvm_subprocess, ppm_utils, scan_results
 
 """
 KVM test definitions.
@@ -274,15 +274,16 @@  def run_autotest(test, params, env):
     cmd += " --exclude=*.pyc"
     cmd += " --exclude=*.svn"
     cmd += " --exclude=*.git"
-    kvm_utils.run_bg(cmd % (test.bindir, tarred_autotest_path), timeout=30)
+    kvm_subprocess.run_fg(cmd % (test.bindir, tarred_autotest_path), timeout=30)
 
     # tar the contents of bindir/autotest/tests/<test_name>
     cmd = "cd %s; tar cvjf %s %s/*"
     cmd += " --exclude=*.pyc"
     cmd += " --exclude=*.svn"
     cmd += " --exclude=*.git"
-    kvm_utils.run_bg(cmd % (os.path.join(test.bindir, "autotest", "tests"),
-                            tarred_test_path, test_name), timeout=30)
+    kvm_subprocess.run_fg(cmd %
+                          (os.path.join(test.bindir, "autotest", "tests"),
+                           tarred_test_path, test_name), timeout=30)
 
     # Check if we need to copy autotest.tar.bz2
     copy = False