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

Proposed by Joe Talbott
Status: Merged
Approved by: Chris Johnston
Approved revision: 706
Merged at revision: 706
Proposed branch: lp://staging/~joetalbott/qa-dashboard/power_chart_fix
Merge into: lp://staging/qa-dashboard
Diff against target: 315 lines (+150/-43)
6 files modified
common/static/js/graph.js (+46/-38)
eventstat/api.py (+0/-3)
power/api.py (+2/-1)
power/management/commands/jenkins_pull_power.py (+5/-0)
power/migrations/0009_remove_nans.py (+93/-0)
qa_dashboard/settings.py (+4/-1)
To merge this branch: bzr merge lp://staging/~joetalbott/qa-dashboard/power_chart_fix
Reviewer Review Type Date Requested Status
Chris Johnston Approve
PS Jenkins bot continuous-integration Approve
Joe Talbott Needs Resubmitting
Andy Doan (community) Approve
Review via email: mp+198758@code.staging.launchpad.net

Commit message

Fix power/eventstat charts for label/column count mismatches.

Description of the change

Fix power/eventstat charts for label/column count mismatches.

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

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

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

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

Error on /power/ amd64 still

 review needs-fixing

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

power - Handle oddities in data files which unbreaks the charts.

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

The errors should now be fixed.

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

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

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

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

Test failure :-(

 review needs-fixing

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

power - Duration isn't always set so handle when it's None.

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

Fixed.

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

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

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

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

merge with trunk

706. By Joe Talbot <email address hidden>

power - Add data migration to convert '-nan' and 'nan' values to acceptable values.

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

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

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/dashboard-ci/284/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