Only write a job keyval out if there isn't already a keyval with
"job_started" in it. This way we don't stomp all over any existing
values when doing secondary follow-up jobs like crashinfo collection.
Risk: Low
Visibility: Prevents secondary jobs from overwriting the keyvals of
the primary job.
Signed-off-by: John Admanski <jadmanski@google.com>
git-svn-id: http://test.kernel.org/svn/autotest/trunk@3024 592f7852-d20e-0410-864c-8624ca9c26a4
diff --git a/server/server_job.py b/server/server_job.py
index 9425530..7fdd9d0 100755
--- a/server/server_job.py
+++ b/server/server_job.py
@@ -162,8 +162,10 @@
'status_version' : str(self.STATUS_VERSION),
'job_started' : str(int(time.time()))}
if self.resultdir:
- job_data.update(get_site_job_data(self))
- utils.write_keyval(self.resultdir, job_data)
+ # only write these keyvals out on the first job in a resultdir
+ if 'job_started' not in utils.read_keyval(self.resultdir):
+ job_data.update(get_site_job_data(self))
+ utils.write_keyval(self.resultdir, job_data)
self.parse_job = parse_job
if self.parse_job and len(machines) == 1: