[KVM-AUTOTEST PATCH] KVM test: kvm_tests: fix a litte bug in timedrift()

[KVM-AUTOTEST PATCH] KVM test: kvm_tests: fix a litte bug in timedrift()

[Cao, Chen]

the original version compared a wrong drift value with given threshold.

Signed-off-by: Cao, Chen <kcao@redhat.com>
---
 client/tests/kvm/kvm_tests.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/client/tests/kvm/kvm_tests.py b/client/tests/kvm/kvm_tests.py
index b100269..4e3391e 100644
--- a/client/tests/kvm/kvm_tests.py
+++ b/client/tests/kvm/kvm_tests.py
@@ -773,7 +773,7 @@ def run_timedrift(test, params, env):
     # Fail the test if necessary
     if drift > drift_threshold:
         raise error.TestFail("Time drift too large: %.2f%%" % drift)
-    if drift > drift_threshold_after_rest:
+    if drift_total > drift_threshold_after_rest:
         raise error.TestFail("Time drift too large after rest period: %.2f%%"
                              % drift_total)
 
-- 
1.6.0.6

Re: [Autotest] [KVM-AUTOTEST PATCH] KVM test: kvm_tests: fix a litte bug in timedrift()

[Lucas Meneghel Rodrigues]

Ok, applied, thanks!

On Fri, Aug 28, 2009 at 6:55 AM, Cao, Chen<kcao@redhat.com> wrote:
> the original version compared a wrong drift value with given threshold.
>
> Signed-off-by: Cao, Chen <kcao@redhat.com>
> ---
>  client/tests/kvm/kvm_tests.py |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/client/tests/kvm/kvm_tests.py b/client/tests/kvm/kvm_tests.py
> index b100269..4e3391e 100644
> --- a/client/tests/kvm/kvm_tests.py
> +++ b/client/tests/kvm/kvm_tests.py
> @@ -773,7 +773,7 @@ def run_timedrift(test, params, env):
>     # Fail the test if necessary
>     if drift > drift_threshold:
>         raise error.TestFail("Time drift too large: %.2f%%" % drift)
> -    if drift > drift_threshold_after_rest:
> +    if drift_total > drift_threshold_after_rest:
>         raise error.TestFail("Time drift too large after rest period: %.2f%%"
>                              % drift_total)
>
> --
> 1.6.0.6
>
> _______________________________________________
> Autotest mailing list
> Autotest@test.kernel.org
> http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
>



-- 
Lucas