Merge lp://staging/~doanac/qa-dashboard/mega-logs into lp://staging/qa-dashboard

Proposed by Andy Doan
Status: Merged
Approved by: Andy Doan
Approved revision: 711
Merged at revision: 710
Proposed branch: lp://staging/~doanac/qa-dashboard/mega-logs
Merge into: lp://staging/qa-dashboard
Diff against target: 131 lines (+38/-19)
2 files modified
smokeng/models.py (+10/-1)
smokeng/views.py (+28/-18)
To merge this branch: bzr merge lp://staging/~doanac/qa-dashboard/mega-logs
Reviewer Review Type Date Requested Status
Joe Talbott Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+202953@code.staging.launchpad.net

Commit message

fix artifact issues with mega-jobs

A jenkins job may now have artifacts that are specific to a testrun that occurred in a mega-job. As the code currently stands, a crash in one test will wind up being counted for all the test runs. This causes us to show 25 crashes instead of just 1. Additionally, we are showing log files for the job and not just for the run. This adds logic to show the right thing when we are dealing with a mega-job.

Description of the change

fix artifact issues with mega-jobs

A jenkins job may now have artifacts that are specific to a testrun that occurred in a mega-job. As the code currently stands, a crash in one test will wind up being counted for all the test runs. This causes us to show 25 crashes instead of just 1. Additionally, we are showing log files for the job and not just for the run. This adds logic to show the right thing when we are dealing with a mega-job.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:711
http://s-jenkins.ubuntu-ci:8080/job/dashboard-ci/290/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/dashboard-ci/290/rebuild

review: Approve (continuous-integration)
Revision history for this message
Joe Talbott (joetalbott) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches