Merge lp://staging/~doanac/qa-dashboard/live-status-base into lp://staging/qa-dashboard

Proposed by Andy Doan
Status: Merged
Approved by: Chris Johnston
Approved revision: 690
Merged at revision: 683
Proposed branch: lp://staging/~doanac/qa-dashboard/live-status-base
Merge into: lp://staging/qa-dashboard
Diff against target: 615 lines (+386/-11)
10 files modified
common/templatetags/dashboard_extras.py (+13/-0)
common/utils.py (+4/-2)
qa_dashboard/settings.py (+2/-2)
smokeng/admin.py (+2/-1)
smokeng/management/commands/jenkins_pull_smokeng.py (+6/-1)
smokeng/migrations/0008_auto__add_field_smokeresult_status.py (+124/-0)
smokeng/models.py (+76/-1)
smokeng/tables.py (+10/-3)
smokeng/tests.py (+144/-1)
smokeng/views.py (+5/-0)
To merge this branch: bzr merge lp://staging/~doanac/qa-dashboard/live-status-base
Reviewer Review Type Date Requested Status
Chris Johnston Approve
PS Jenkins bot continuous-integration Approve
Joe Talbott Approve
Review via email: mp+196167@code.staging.launchpad.net

Commit message

This provides the base model and view updates required for live-status of smoke jobs. It adds a new "status" field to results so we can see if they are:

 queued: waiting for jenkins to run the test
 running: jenkins is running the test
 syncing: the test is complete with results, but hasn't made it to the public jenkins (and pulled by our scripts)
 complete: its gone through the pull-script and we have all its job artifacts

Description of the change

This provides the base model and view updates required for live-status of smoke jobs. It adds a new "status" field to results so we can see if they are:

 queued: waiting for jenkins to run the test
 running: jenkins is running the test
 syncing: the test is complete with results, but hasn't made it to the public jenkins (and pulled by our scripts)
 complete: its gone through the pull-script and we have all its job artifacts

To post a comment you must log in.
Revision history for this message
Andy Doan (doanac) wrote :

it might be easier reading commit by commit instead of the full diff.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

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

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

review: Approve (continuous-integration)
Revision history for this message
Chris Johnston (cjohnston) wrote :

I don't really have any issues.. I wouldn't mind http://paste.ubuntu.com/6471885/ being applied though

Revision history for this message
Joe Talbott (joetalbott) wrote :

+1

review: Approve
690. By Andy Doan

add admin filtering as per chris

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

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

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

review: Approve (continuous-integration)
Revision history for this message
Chris Johnston (cjohnston) :
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