On 07/12/2012 08:44 AM, Gerd Hoffmann wrote:
> Signed-off-by: Gerd Hoffmann<kra...@redhat.com>
> ---
> client/virt/tests/unattended_install.py | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/client/virt/tests/unattended_install.py
> b/client/virt/tests/unattended_install.py
> index f8f680d..ea8a7c6 100644
> --- a/client/virt/tests/unattended_install.py
> +++ b/client/virt/tests/unattended_install.py
> @@ -866,6 +866,8 @@ class UnattendedInstallConfig(object):
> # it's only necessary to download kernel/initrd if running bare qemu
> if self.vm_type == 'kvm':
> error.context("downloading vmlinuz/initrd.img from %s" %
> self.url)
> + if not os.path.exists(self.image_path):
> + os.mkdir(self.image_path)
> os.chdir(self.image_path)
> kernel_cmd = "wget -q %s/%s/%s" % (self.url,
> self.boot_path,
This looks good to me. Do you think it would make sense to just do this
from setup() and make the assumption that a redundant image_path (if
created) isn't consequential since it's under /tmp/... ?
--
Chris Evich, RHCA, RHCE, RHCDS, RHCSS
Quality Assurance Engineer
e-mail: cevich + `@' + redhat.com o: 1-888-RED-HAT1 x44214
_______________________________________________
Autotest mailing list
Autotest@test.kernel.org
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
