Merge lp://staging/~canonical-platform-qa/qakit/add-vis into lp://staging/qakit

Proposed by Allan LeSage
Status: Merged
Approved by: Allan LeSage
Approved revision: 117
Merged at revision: 127
Proposed branch: lp://staging/~canonical-platform-qa/qakit/add-vis
Merge into: lp://staging/qakit
Diff against target: 1346 lines (+1273/-0)
14 files modified
qakit/vis/dashboard/main.js (+55/-0)
qakit/vis/dashboard/ust_rc-proposed_arale_regression.html (+48/-0)
qakit/vis/dashboard/ust_rc-proposed_arale_sanity.html (+48/-0)
qakit/vis/dashboard/ust_rc-proposed_krillin_regression.html (+48/-0)
qakit/vis/dashboard/ust_rc-proposed_krillin_sanity.html (+48/-0)
qakit/vis/dashboard/ust_rc_arale_regression.html (+48/-0)
qakit/vis/dashboard/ust_rc_arale_sanity.html (+48/-0)
qakit/vis/dashboard/ust_rc_krillin_regression.html (+48/-0)
qakit/vis/dashboard/ust_rc_krillin_sanity.html (+48/-0)
qakit/vis/jenkins_results.py (+70/-0)
qakit/vis/retrieve_jenkins_results.py (+229/-0)
qakit/vis/tests/test_retrieve_jenkins_results.py (+47/-0)
qakit/vis/tests/test_visualize.py (+194/-0)
qakit/vis/visualize.py (+294/-0)
To merge this branch: bzr merge lp://staging/~canonical-platform-qa/qakit/add-vis
Reviewer Review Type Date Requested Status
Canonical Platform QA Team Pending
Review via email: mp+299606@code.staging.launchpad.net

Commit message

Add a vis tool to visualize test suite performance.

Description of the change

Add a 'vis' tool for visualizing test suite status.

Presently we're using a few html files to present this datatables/bootstrap version, could be nicer with tabs but we'll save that for a later version.

To post a comment you must log in.
Revision history for this message
Sergio Cazzolato (sergio-j-cazzolato) wrote :

I made a high level review, I left some comments inline.

113. By Allan LeSage

Move dashboard-type files to a dashboard dir.

114. By Allan LeSage

Context manager for json writer.

115. By Allan LeSage

Use statistics to compute mean.

116. By Allan LeSage

Move to dao, fix tests.

117. By Allan LeSage

Typo!

Revision history for this message
Allan LeSage (allanlesage) wrote :

Made all the requested changes, thanks cachio for a further review!

Revision history for this message
Sergio Cazzolato (sergio-j-cazzolato) wrote :

Organized now! go ahead with this change

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

to all changes: