LGTM

On Thu, Nov 8, 2012 at 6:39 PM, Chris Evich <[email protected]> wrote:
> Noticed that aexpect drops all it's pipes and output
> files under /tmp/kvm_spawn (for virt tests).  However,
> it never cleans them up.  Deploy the nuclear option,
> and ignore errors that get in the way.
>
> Signed-off-by: Chris Evich <[email protected]>
> ---
>  client/tests            |    2 +-
>  client/tools/make_clean |    5 ++++-
>  2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/client/tests b/client/tests
> index 247d18a..e641069 160000
> --- a/client/tests
> +++ b/client/tests
> @@ -1 +1 @@
> -Subproject commit 247d18a547fa4138124885e39000b2163cca8d7e
> +Subproject commit e641069f5b8b85bf590abb97feab9627459ffd4e
> diff --git a/client/tools/make_clean b/client/tools/make_clean
> index 267bed0..fdc4a03 100755
> --- a/client/tools/make_clean
> +++ b/client/tools/make_clean
> @@ -1,5 +1,5 @@
>  #!/usr/bin/python
> -import os, sys
> +import os, sys, shutil
>
>  def purge_src(top_dir):
>      if not os.path.exists(top_dir):
> @@ -40,6 +40,9 @@ if __name__ == '__main__':
>                     'tests/virt']:
>          purge_src(subdir)
>
> +    if os.path.isdir('/tmp/kvm_spawn'):
> +        shutil.rmtree('/tmp/kvm_spawn', ignore_errors=True)
> +
>      for filename in ['/tmp/address_pool', '/tmp/address_pool.lock',
>                       '/tmp/kvm-autotest-vm-create.lock',
>                       '/tmp/libvirt-autotest-vm-create.lock',
> --
> 1.7.1
>
> _______________________________________________
> Autotest-kernel mailing list
> [email protected]
> https://www.redhat.com/mailman/listinfo/autotest-kernel



-- 
Lucas

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

Reply via email to