Merge lp:~bladernr/checkbox-certification/cert-add-cert-whitelist into lp:checkbox-certification

Proposed by Jeff Lane 
Status: Merged
Merge reported by: Marc Tardif
Merged at revision: not available
Proposed branch: lp:~bladernr/checkbox-certification/cert-add-cert-whitelist
Merge into: lp:checkbox-certification
Diff against target: 90 lines (+12/-10)
5 files modified
jobs/certification.txt (+1/-1)
jobs/cpu.txt (+5/-3)
jobs/hardware.txt (+2/-2)
jobs/kernel.txt (+2/-2)
jobs/sru_suite.txt (+2/-2)
To merge this branch: bzr merge lp:~bladernr/checkbox-certification/cert-add-cert-whitelist
Reviewer Review Type Date Requested Status
Marc Tardif (community) Approve
Review via email: mp+48270@code.launchpad.net

Description of the change

Certification Whitelist In tha HOUSE.

To post a comment you must log in.
Revision history for this message
Marc Tardif (cr3) wrote :

Any particular reason why the acpi_sleep job was renamed to acpi_sleep_attachment? For example, was it conflicting with another job?

review: Needs Information
Revision history for this message
Jeff Lane  (bladernr) wrote :

On 02/02/2011 12:18 PM, Marc Tardif wrote:
> Review: Needs Information
> Any particular reason why the acpi_sleep job was renamed to acpi_sleep_attachment? For example, was it conflicting with another job?

actually, I forgot to change the rest of them...

That one was changed for consistency sake, since I also changed all the
attachment jobs in base to *_attach I meant to change all the ones in
-cert to match. I failed that bit of consistency as it was a last
minute addition while playing with the whitelist...

It does not currently conflict with any actual test job, however, who
knows what could happen in the future... I found conflicts in base,
which prompted the renaming...

--
Jeff Lane - Hardware Certification Engineer and Test Tools Developer
Ubuntu Ham: W4KDH
Freenode IRC: bladernr or bladernr_
gpg: 1024D/3A14B2DD 8C88 B076 0DD7 B404 1417 C466 4ABD 3635 3A14 B2DD

325. By Jeff Lane 

Created seperate whitelists for client and server certification

Revision history for this message
Marc Tardif (cr3) wrote :

Looks good and took the opportunity to rename a few jobs:

sru_attach_screenshot -> sru_screenshot_attachment
sru_xorg-failsafe -> sru_xorg_failsafe
sru_xorg-process -> sru_xorg_process
sru_fwts-wakealarm -> sru_fwts_wakealarm

And made sure to update the whitelists accordingly.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'jobs/certification.txt'
2--- jobs/certification.txt 2011-01-12 21:19:16 +0000
3+++ jobs/certification.txt 2011-02-02 20:04:26 +0000
4@@ -4,7 +4,7 @@
5 description: Certification tests.
6
7 plugin: attachment
8-name: acpi_sleep
9+name: acpi_sleep_attachment
10 command: cat /proc/acpi/sleep
11
12 plugin: local
13
14=== modified file 'jobs/cpu.txt'
15--- jobs/cpu.txt 2010-08-27 22:30:33 +0000
16+++ jobs/cpu.txt 2011-02-02 20:04:26 +0000
17@@ -1,10 +1,12 @@
18 plugin: shell
19 name: cpu_scaling_test
20-requires: package.name == 'linux'
21+requires:
22+ package.name == 'linux'
23+ package.name == 'fwts'
24 user: root
25-command: cpu_scaling_test
26+command: fwts_test -c -l $CHECKBOX_DATA/cpu_scaling_test.log
27 description:
28- Test the CPU scaling capabilities.
29+ Test the CPU scaling capabilities using Colin King's Firmware Test Suite tool.
30
31 plugin: shell
32 name: clocktest
33
34=== modified file 'jobs/hardware.txt'
35--- jobs/hardware.txt 2010-12-03 22:38:58 +0000
36+++ jobs/hardware.txt 2011-02-02 20:04:26 +0000
37@@ -14,7 +14,7 @@
38 description:
39 Run Colin King's Firmware Test Suite automated tests.
40 command:
41- fwts_test -l $CHECKBOX_CERTIFICATION_SHARE/fwts_results.log
42+ fwts_test -l $CHECKBOX_DATA/fwts_results.log
43
44 plugin: attachment
45 name: fwts_log
46@@ -22,4 +22,4 @@
47 description:
48 Gather log from the firmware test suite run
49 command:
50- cat $CHECKBOX_CERTIFICATION_SHARE/fwts_results.log
51+ cat $CHECKBOX_DATA/fwts_results.log
52
53=== modified file 'jobs/kernel.txt'
54--- jobs/kernel.txt 2010-12-03 22:38:58 +0000
55+++ jobs/kernel.txt 2011-02-02 20:04:26 +0000
56@@ -1,10 +1,10 @@
57 plugin: shell
58-name: fwts-wakealarm
59+name: fwts_wakealarm
60 description: Run Colin Kings FWTS wakealarm test
61 requires:
62 package.name == 'linux'
63 package.name == 'fwts'
64-command: fwts_test -w -l $CHECKBOX_CERTIFICATION_SHARE/fwts-wakealarm.log
65+command: fwts_test -w -l $CHECKBOX_DATA/fwts-wakealarm.log
66
67 plugin: shell
68 name: tickless_idle_test
69
70=== modified file 'jobs/sru_suite.txt'
71--- jobs/sru_suite.txt 2011-01-31 15:54:46 +0000
72+++ jobs/sru_suite.txt 2011-02-02 20:04:26 +0000
73@@ -78,7 +78,7 @@
74 package.name == 'linux'
75 package.name == 'fwts'
76 user: root
77-command: fwts_test -c -l $CHECKBOX_CERTIFICATION_SHARE/fwts_results.log
78+command: fwts_test -c -l $CHECKBOX_DATA/fwts_results.log
79 description:
80 Test the CPU scaling capabilities.
81
82@@ -90,7 +90,7 @@
83 description:
84 Run Colin King's FWTS wakealarm test
85 command:
86- fwts_test -w -l $CHECKBOX_CERTIFICATION_SHARE/fwts_results.log
87+ fwts_test -w -l $CHECKBOX_DATA/fwts_results.log
88
89 plugin: shell
90 name: sru_tickless_idle_test

Subscribers

People subscribed via source and target branches