Modify autotest to support preserving minidumps generated during tests.

server/server_job.py

Index: server/server_job.py
diff --git a/server/server_job.py b/server/server_job.py
index 8a16c6a4fdb06ba250149d459d29cfd9c34ccd30..3e6826b2ce3b9282a63760cb05b0950fab2f08c9 100644
--- a/server/server_job.py
+++ b/server/server_job.py
@@ -37,17 +37,6 @@ VERIFY_CONTROL_FILE = _control_segment_path('verify')
 REPAIR_CONTROL_FILE = _control_segment_path('repair')
 
 
-# by default provide a stub that generates no site data
-def _get_site_job_data_dummy(job):
-    return {}
-
-
-# load up site-specific code for generating site-specific job data
-get_site_job_data = utils.import_site_function(__file__,
-    "autotest_lib.server.site_server_job", "get_site_job_data",
-    _get_site_job_data_dummy)
-
-
 class status_indenter(base_job.status_indenter):
     """Provide a simple integer-backed status indenter."""
     def __init__(self):
@@ -1108,6 +1097,17 @@ class base_server_job(base_job.base_job):
                 host.clear_known_hosts()
 
 
+# by default provide a stub that generates no site data

[kmixter1, 2011/02/23 02:00:39]

was this just moved?  Were you just moving it to get it closer to point of use?

[thieule, 2011/03/03 01:27:41]

This block needed to be moved down because it imports site_server_job. 
site_server_job.py relies on base_server_job being declared.  With this block
being higher up in the file, base_server_job has not been declared yet.

Anyways, no need for this anymore since we're using site_crashcollector instead
of site_server_job.

+def _get_site_job_data_dummy(job):
+    return {}
+
+
+# load up site-specific code for generating site-specific job data
+get_site_job_data = utils.import_site_function(__file__,
+    "autotest_lib.server.site_server_job", "get_site_job_data",
+    _get_site_job_data_dummy)
+
+
 site_server_job = utils.import_site_class(
     __file__, "autotest_lib.server.site_server_job", "site_server_job",
     base_server_job)