Merge ~alanec/plainbox-provider-checkbox:jobs_cpufreq_cpuinfo into plainbox-provider-checkbox:master

Proposed by Adrian Lane
Status: Merged
Approved by: Adrian Lane
Approved revision: 1b710fd85006c830cd69bbb18629a8ce719d2a7f
Merged at revision: 66ec706a40dbbbf31d8c8c766513819b5e89657c
Proposed branch: ~alanec/plainbox-provider-checkbox:jobs_cpufreq_cpuinfo
Merge into: plainbox-provider-checkbox:master
Diff against target: 15 lines (+2/-0)
1 file modified
units/cpu/jobs.pxu (+2/-0)
Reviewer Review Type Date Requested Status
Jonathan Cave (community) Approve
Review via email: mp+400086@code.launchpad.net

Commit message

Updated cpu/cpufreq_test-server to require cpuinfo.scaling == 'supported'; added est test duration.

To post a comment you must log in.
Revision history for this message
Jonathan Cave (jocave) wrote :

+1, thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/units/cpu/jobs.pxu b/units/cpu/jobs.pxu
2index afdd6ea..b3167c4 100644
3--- a/units/cpu/jobs.pxu
4+++ b/units/cpu/jobs.pxu
5@@ -181,8 +181,10 @@ _siblings: [
6 plugin: shell
7 category_id: com.canonical.plainbox::cpu
8 id: cpu/cpufreq_test-server
9+estimated_duration: 3.0
10 user: root
11 command: cpufreq_test.py -q
12+requires: cpuinfo.scaling == 'supported'
13 _summary:
14 cpufreq scaling test
15 _description:

Subscribers

People subscribed via source and target branches