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
=== modified file 'jobs/certification.txt'
--- jobs/certification.txt 2011-01-12 21:19:16 +0000
+++ jobs/certification.txt 2011-02-02 20:04:26 +0000
@@ -4,7 +4,7 @@
4description: Certification tests.4description: Certification tests.
55
6plugin: attachment6plugin: attachment
7name: acpi_sleep7name: acpi_sleep_attachment
8command: cat /proc/acpi/sleep8command: cat /proc/acpi/sleep
99
10plugin: local10plugin: local
1111
=== modified file 'jobs/cpu.txt'
--- jobs/cpu.txt 2010-08-27 22:30:33 +0000
+++ jobs/cpu.txt 2011-02-02 20:04:26 +0000
@@ -1,10 +1,12 @@
1plugin: shell1plugin: shell
2name: cpu_scaling_test2name: cpu_scaling_test
3requires: package.name == 'linux'3requires:
4 package.name == 'linux'
5 package.name == 'fwts'
4user: root6user: root
5command: cpu_scaling_test7command: fwts_test -c -l $CHECKBOX_DATA/cpu_scaling_test.log
6description:8description:
7 Test the CPU scaling capabilities.9 Test the CPU scaling capabilities using Colin King's Firmware Test Suite tool.
810
9plugin: shell11plugin: shell
10name: clocktest12name: clocktest
1113
=== modified file 'jobs/hardware.txt'
--- jobs/hardware.txt 2010-12-03 22:38:58 +0000
+++ jobs/hardware.txt 2011-02-02 20:04:26 +0000
@@ -14,7 +14,7 @@
14description:14description:
15 Run Colin King's Firmware Test Suite automated tests.15 Run Colin King's Firmware Test Suite automated tests.
16command:16command:
17 fwts_test -l $CHECKBOX_CERTIFICATION_SHARE/fwts_results.log17 fwts_test -l $CHECKBOX_DATA/fwts_results.log
1818
19plugin: attachment19plugin: attachment
20name: fwts_log20name: fwts_log
@@ -22,4 +22,4 @@
22description:22description:
23 Gather log from the firmware test suite run23 Gather log from the firmware test suite run
24command:24command:
25 cat $CHECKBOX_CERTIFICATION_SHARE/fwts_results.log25 cat $CHECKBOX_DATA/fwts_results.log
2626
=== modified file 'jobs/kernel.txt'
--- jobs/kernel.txt 2010-12-03 22:38:58 +0000
+++ jobs/kernel.txt 2011-02-02 20:04:26 +0000
@@ -1,10 +1,10 @@
1plugin: shell1plugin: shell
2name: fwts-wakealarm2name: fwts_wakealarm
3description: Run Colin Kings FWTS wakealarm test3description: Run Colin Kings FWTS wakealarm test
4requires:4requires:
5 package.name == 'linux'5 package.name == 'linux'
6 package.name == 'fwts'6 package.name == 'fwts'
7command: fwts_test -w -l $CHECKBOX_CERTIFICATION_SHARE/fwts-wakealarm.log7command: fwts_test -w -l $CHECKBOX_DATA/fwts-wakealarm.log
88
9plugin: shell9plugin: shell
10name: tickless_idle_test10name: tickless_idle_test
1111
=== modified file 'jobs/sru_suite.txt'
--- jobs/sru_suite.txt 2011-01-31 15:54:46 +0000
+++ jobs/sru_suite.txt 2011-02-02 20:04:26 +0000
@@ -78,7 +78,7 @@
78 package.name == 'linux'78 package.name == 'linux'
79 package.name == 'fwts'79 package.name == 'fwts'
80user: root80user: root
81command: fwts_test -c -l $CHECKBOX_CERTIFICATION_SHARE/fwts_results.log81command: fwts_test -c -l $CHECKBOX_DATA/fwts_results.log
82description:82description:
83 Test the CPU scaling capabilities.83 Test the CPU scaling capabilities.
8484
@@ -90,7 +90,7 @@
90description:90description:
91 Run Colin King's FWTS wakealarm test91 Run Colin King's FWTS wakealarm test
92command:92command:
93 fwts_test -w -l $CHECKBOX_CERTIFICATION_SHARE/fwts_results.log93 fwts_test -w -l $CHECKBOX_DATA/fwts_results.log
9494
95plugin: shell95plugin: shell
96name: sru_tickless_idle_test96name: sru_tickless_idle_test

Subscribers

People subscribed via source and target branches