summaryrefslogtreecommitdiff
path: root/lkft
diff options
context:
space:
mode:
authorYongqin Liu <yongqin.liu@linaro.org>2019-08-21 03:16:29 +0800
committerYongqin Liu <yongqin.liu@linaro.org>2019-08-21 03:16:29 +0800
commitc4a1f8e9e3d30f7356205cba5672a54511bfaf45 (patch)
treeba3323c9459f8e1b25457d9e501ea43ed69061d7 /lkft
parentf353a7ed3d88c2682d74019d9b31a5bb0070abcb (diff)
lkft/view.py: fix the problem to show wrong numbers for builds
resubmitted jobs should not be accounted any more Signed-off-by: Yongqin Liu <yongqin.liu@linaro.org>
Diffstat (limited to 'lkft')
-rw-r--r--lkft/views.py11
1 files changed, 10 insertions, 1 deletions
diff --git a/lkft/views.py b/lkft/views.py
index bffb238..4ae7a55 100644
--- a/lkft/views.py
+++ b/lkft/views.py
@@ -242,10 +242,18 @@ def list_builds(request):
if number_of_build_with_jobs < BUILD_WITH_JOBS_NUMBER:
jobs = qa_report_api.get_jobs_for_build(build.get('id'))
+
+ resubmitted_job_urls = [ job.get('parent_job') for job in jobs if job.get('parent_job')]
download_attachments_save_result(jobs=jobs)
+ job_names = []
for job in jobs:
- if job.get('parent_job'):
+ if job.get('url') in resubmitted_job_urls:
# ignore jobs which were resubmitted
+ logger.info("%s: %s:%s has been resubmitted already" % (build.get('version'), job.get('job_id'), job.get('url')))
+ continue
+
+ if job.get('name') in job_names:
+ logger.info("%s %s: %s %s the same name job has been recorded" % (build.get('version'), job.get('name'), job.get('job_id'), job.get('url')))
continue
def get_testcases_number_for_job(job):
@@ -283,6 +291,7 @@ def list_builds(request):
build_modules_total = build_modules_total + numbers.get('modules_total')
build_modules_done = build_modules_done + numbers.get('modules_done')
job['numbers'] = numbers
+ job_names.append(job.get('name'))
number_of_build_with_jobs = number_of_build_with_jobs + 1
build['numbers'] = {
'number_passed': build_number_passed,