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 05/21/2011 02:59 AM, Lucas Meneghel Rodrigues wrote:
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!
Thanks for your comment.
After merge upstream code to our local tree, I found that our local unattended_install could not work. Before, Current working folder for unattended.py is kvm folder, changed in process_command.
Now  Current working folder for

    unattended_install_config = UnattendedInstallConfig(test, params)
    unattended_install_config.setup()

changed to case result folder. So our unattended_install always fails at could not find ks.iso.
Then I send this patch.

I will recheck our local code and configure.
Thanks very much!

Feng Yang
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

--
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