Merge lp://staging/~toykeeper/ubuntu-power-tests/music-video-lowpower-statistics into lp://staging/ubuntu-power-tests

Proposed by Selene ToyKeeper
Status: Merged
Merged at revision: 16
Proposed branch: lp://staging/~toykeeper/ubuntu-power-tests/music-video-lowpower-statistics
Merge into: lp://staging/ubuntu-power-tests
Diff against target: 691 lines (+482/-24)
12 files modified
README (+5/-1)
adt/debian/tests/control (+4/-1)
adt/debian/tests/power_usage_idle_reduced (+28/-0)
adt/debian/tests/power_usage_music (+126/-0)
adt/debian/tests/power_usage_video (+95/-0)
adt/debian/tests/prepare-device (+4/-0)
detect_regressions.py (+173/-0)
flash/flash.sh (+9/-8)
flash/media/urls (+2/-0)
power-graph.py (+6/-1)
power.py (+25/-8)
test-new-builds.sh (+5/-5)
To merge this branch: bzr merge lp://staging/~toykeeper/ubuntu-power-tests/music-video-lowpower-statistics
Reviewer Review Type Date Requested Status
Max Brustkern (community) Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+270013@code.staging.launchpad.net

Commit message

Pulled in all recent unmerged changes: music test, video test, idle_reduced test, and a script to detect anomalies in recent results.

Description of the change

Pulled in all recent unmerged changes: music test, video test, idle_reduced test, and a script to detect anomalies in recent results.

To post a comment you must log in.
Revision history for this message
Selene ToyKeeper (toykeeper) wrote :

This MP obsoletes an earlier one which never finished merging: https://code.launchpad.net/~toykeeper/ubuntu-power-tests/add-music-video-tests/+merge/266043

The missing debian directory is still an open issue; I'm not sure how much of that adt needs and how much CI needs, so I haven't resolved it yet. I'd prefer to avoid duplicating that information if possible. With the adt tests in the proper debian directory, adt was pushing *way* too much data to the device under test, and actually causing the tests to time out or otherwise fail. So, those need to be somewhere else. This makes the packaging a bit awkward.

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

Maybe creating a basic debian directory that applies to the actual package would sort things out? I can work on that if you want. Other than that, I think maybe leaving out some of the stuff that's been commented out would make sense, but it looks good to me.

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