Re: [Autotest] [PATCH] KVM Test: Switch current working folder in unattended_install.py.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2011-05-19 at 18:24 +0800, fyang@xxxxxxxxxx wrote:
> From: Feng Yang <fyang@xxxxxxxxxx>
> 
> Current working folder for
>     unattended_install_config = UnattendedInstallConfig(test, params)
>     unattended_install_config.setup()
> must be kvm folder.

This is not needed at all. What might be going on your setup is some
incorrectly set or absent path that might be messing up with relative
paths during your install.

Please provide some more info so we can fix your problem properly.
Meanwhile I'm marking this as 'rejected'.

Thanks!

> Signed-off-by: Feng Yang <fyang@xxxxxxxxxx>
> ---
>  client/tests/kvm/tests/unattended_install.py |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/client/tests/kvm/tests/unattended_install.py b/client/tests/kvm/tests/unattended_install.py
> index 50a8c7a..eee1761 100644
> --- a/client/tests/kvm/tests/unattended_install.py
> +++ b/client/tests/kvm/tests/unattended_install.py
> @@ -506,8 +506,11 @@ def run_unattended_install(test, params, env):
>      @param params: Dictionary with the test parameters.
>      @param env: Dictionary with test environment.
>      """
> +    cur_folder = os.getcwd()
> +    os.chdir(test.bindir)
>      unattended_install_config = UnattendedInstallConfig(test, params)
>      unattended_install_config.setup()
> +    os.chdir(cur_folder)
>      vm = env.get_vm(params["main_vm"])
>      vm.create()
>  


--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux