Merge lp://staging/~joetalbott/qa-dashboard/suspend_blockers into lp://staging/qa-dashboard

Proposed by Joe Talbott
Status: Merged
Approved by: Joe Talbott
Approved revision: 759
Merged at revision: 753
Proposed branch: lp://staging/~joetalbott/qa-dashboard/suspend_blockers
Merge into: lp://staging/qa-dashboard
Diff against target: 1659 lines (+1507/-3)
19 files modified
plugins.py (+2/-1)
qa_dashboard/settings.py (+2/-1)
scripts/ci-dashboard-update.py (+7/-1)
suspend/api.py (+93/-0)
suspend/dashboard.py (+52/-0)
suspend/management/commands/jenkins_pull_suspend.py (+306/-0)
suspend/migrations/0001_initial.py (+193/-0)
suspend/models.py (+68/-0)
suspend/static/suspend/graph.js (+53/-0)
suspend/static/suspend/style.css (+5/-0)
suspend/tables.py (+73/-0)
suspend/templates/suspend/arch_overview.html (+75/-0)
suspend/templates/suspend/image_overview.html (+47/-0)
suspend/templates/suspend/suspendimage_detail.html (+120/-0)
suspend/tests.py (+167/-0)
suspend/urls.py (+30/-0)
suspend/urls_api.py (+35/-0)
suspend/utils.py (+61/-0)
suspend/views.py (+118/-0)
To merge this branch: bzr merge lp://staging/~joetalbott/qa-dashboard/suspend_blockers
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Chris Johnston Approve
Review via email: mp+224861@code.staging.launchpad.net

Commit message

Description of the change

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

FAILED: Continuous integration, rev:748
http://s-jenkins.ubuntu-ci:8080/job/dashboard-ci/330/
Executed test runs:

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

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

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

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

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

I don't see where this is being added to the pull script cron.. Is that missing?

It would be nice if we could test the different URLs as we do in other places (smokeng) just to ensure they return 200.

Also, when I tried to import data, nothing actually imported. Am I missing something there?

review: Needs Fixing
749. By Joe Talbot <email address hidden>

suspend - Add to update scripts

750. By Joe Talbot <email address hidden>

suspend - Follow style guidelines and only capitalize the first character of headings

751. By Joe Talbot <email address hidden>

suspend - Add more tests

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

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

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

review: Approve (continuous-integration)
752. By Joe Talbot <email address hidden>

suspend - Limit graphs to 60 items

753. By Joe Talbot <email address hidden>

suspend - Make graph lines more narrow

754. By Joe Talbot <email address hidden>

suspend - Fix table sorting for image build_numbers.

755. By Joe Talbot <email address hidden>

suspend - Fix buildnumber sorting

756. By Joe Talbot <email address hidden>

suspend - Fix some minor style issues.

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

FAILED: Continuous integration, rev:756
http://s-jenkins.ubuntu-ci:8080/job/dashboard-ci/339/
Executed test runs:

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

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

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

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

review: Approve (continuous-integration)
757. By Joe Talbot <email address hidden>

update copyrights

758. By Joe Talbot <email address hidden>

suspend - remove unneeded style code

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

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

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

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

Looks fine.. thanks for the fixes!

review: Approve
759. By Joe Talbot <email address hidden>

suspend - Add another digit of precision to percentages.

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

FAILED: Continuous integration, rev:759
http://s-jenkins.ubuntu-ci:8080/job/dashboard-ci/345/
Executed test runs:

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

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

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

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

review: Approve (continuous-integration)

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