Lucas Meneghel Rodrigues | 27 Jun 2012 01:37
Picon
Favicon
Gravatar

[PATCH] server.autotest_remote: Transfer and clean state files properly

The autotest_remote code assumes the state of remote
control files is simply path/to/control + ".state". This
causes problem with our patches moving state file creation
to the test output dir set on global_config.ini, with
fall back to the autotest tmpdir area.

Fix the code, creating new attributes for the _BaseRun
class, that will make it easier to control and keep
consistency of the state file locations.

Signed-off-by: Lucas Meneghel Rodrigues <lmr <at> redhat.com>
---
 server/autotest_remote.py |   22 +++++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

diff --git a/server/autotest_remote.py b/server/autotest_remote.py
index d092a04..8d017d0 100644
--- a/server/autotest_remote.py
+++ b/server/autotest_remote.py
 <at>  <at>  -348,9 +348,9  <at>  <at>  class BaseAutotest(installable_object.InstallableObject):
             pass

         delete_file_list = [atrun.remote_control_file,
-                            atrun.remote_control_file + '.state',
+                            atrun.remote_control_state,
                             atrun.manual_control_file,
-                            atrun.manual_control_file + '.state']
+                            atrun.manual_control_state]
         cmd = ';'.join('rm -f ' + control for control in delete_file_list)
         host.run(cmd, ignore_status=True)
(Continue reading)


Gmane