On Fri, 2012-08-03 at 15:28 +0200, Mackowski Rafal wrote:
> time.sleep(seconds) was executed on server not a client.
> After changes client execute sleep and server wait
> until client finish the job.

Great, makes a ton of sense, applied. I had the same problem applying
this patch (patch scrambled), please verify your future patches.

> Signed-off-by: Rafal Mackowski <[email protected]>
> ---
>   server/tests/sleeptest/control      |    2 +-
>   server/tests/sleeptest/sleeptest.py |    5 ++---
>   2 files changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/server/tests/sleeptest/control b/server/tests/sleeptest/control
> index f417729..cb21907 100644
> --- a/server/tests/sleeptest/control
> +++ b/server/tests/sleeptest/control
> @@ -12,6 +12,6 @@ runs sleep for one second on the list of machines.
> 
>   def run(machine):
>       host = hosts.create_host(machine)
> -    job.run_test('sleeptest')
> +    job.run_test('sleeptest', host=host)
> 
>   job.parallel_simple(run, machines)
> diff --git a/server/tests/sleeptest/sleeptest.py 
> b/server/tests/sleeptest/sleeptest.py
> index 4109439..56663b7 100644
> --- a/server/tests/sleeptest/sleeptest.py
> +++ b/server/tests/sleeptest/sleeptest.py
> @@ -1,8 +1,7 @@
> -import time
>   from autotest.server import test
> 
>   class sleeptest(test.test):
>       version = 1
> 
> -    def run_once(self, host=None, seconds=1):
> -        time.sleep(seconds)
> +    def execute(self, host, seconds=1):
> +        host.run('sleep ' + str(seconds))


_______________________________________________
Autotest-kernel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/autotest-kernel

Reply via email to