Re: [KVM-AUTOTEST][PATCH] Enable running test(s) multiple times (iterations)

From: Lucas Meneghel Rodrigues <lmr@redhat.com>
To: Uri Lublin <uril@redhat.com>
Cc: kvm@vger.kernel.org, autotest@test.kernel.org, supriyak@in.ibm.com
Subject: Re: [KVM-AUTOTEST][PATCH] Enable running test(s) multiple times (iterations)
Date: Fri, 12 Jun 2009 16:43:36 -0300	[thread overview]
Message-ID: <1244835816.2822.111.camel@localhost.localdomain> (raw)
In-Reply-To: <1244718925-24670-2-git-send-email-uril@redhat.com>

On Thu, 2009-06-11 at 14:15 +0300, Uri Lublin wrote:
> From: Supriya Kannery <supriyak@in.ibm.com>
> 
> Default is to run each test once.
> 
> Just add iterations = N in kvm_tests.cfg to the test(s) you
> want to run multiple times.
> 
> Signed-off-by: Supriya Kannery <supriyak@in.ibm.com>
> Cc : Michael Goldish <mgoldish@redhat.com>
> Signed-off-by: Uri Lublin <uril@redhat.com>
> ---
>  client/tests/kvm/control |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/client/tests/kvm/control b/client/tests/kvm/control
> index b3543ee..c030a14 100644
> --- a/client/tests/kvm/control
> +++ b/client/tests/kvm/control
> @@ -145,8 +145,10 @@ for dict in list:
>                  dependencies_satisfied = False
>                  break
>      if dependencies_satisfied:
> +        test_iterations=int(dict.get("iterations", 1))
>          current_status = job.run_test("kvm", params=dict,
> -                                      tag=dict.get("shortname"))
> +                                      tag=dict.get("shortname"),
> +                                      iterations=test_iterations)
>      else:
>          current_status = False
>      status_dict[dict.get("name")] = current_status

Looks good to me, applied. Thanks!

-- 
Lucas Meneghel Rodrigues
Software Engineer (QE)
Red Hat - Emerging Technologies