Merge lp:~brendan-donegan/checkbox/bug974271_udev_resource_attachment into lp:checkbox

Proposed by Brendan Donegan
Status: Merged
Merged at revision: 1638
Proposed branch: lp:~brendan-donegan/checkbox/bug974271_udev_resource_attachment
Merge into: lp:checkbox
Diff against target: 27 lines (+6/-0)
2 files modified
debian/changelog (+1/-0)
jobs/info.txt.in (+5/-0)
To merge this branch: bzr merge lp:~brendan-donegan/checkbox/bug974271_udev_resource_attachment
Reviewer Review Type Date Requested Status
Marc Tardif (community) Approve
Review via email: mp+122653@code.launchpad.net

Description of the change

An attachment job was requested to store the output of udev_resource for situations where a requirement fails and it is not clear why. Seeing the output of udev_resource will help debugging this. This is in aid of self-testing, since we may not have access to the physical machine.

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

It's unfortunate that we've had to come to attaching so many things but it's necessary. This will help troubleshoot some skipped tests, thanks!

review: Approve
Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

We could have a review of the current attachment jobs and see if any of them are unused.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2012-09-04 09:15:17 +0000
3+++ debian/changelog 2012-09-04 10:44:52 +0000
4@@ -51,6 +51,7 @@
5 * [FEATURE] Added semi-automated wireless tests which require only a single
6 router to run, prompting the user to modify the routers config during
7 the test.
8+ * Attach the output of udev_resource, for debugging purposes (LP: #974271)
9
10 -- Jeff Lane <jeff@ubuntu.com> Fri, 31 Aug 2012 13:30:24 -0400
11
12
13=== modified file 'jobs/info.txt.in'
14--- jobs/info.txt.in 2012-08-29 21:09:30 +0000
15+++ jobs/info.txt.in 2012-09-04 10:44:52 +0000
16@@ -82,6 +82,11 @@
17 command: udevadm info --export-db
18 _description: Attaches a dump of the udev database showing system hardware information.
19
20+name: udev_resource_attachment
21+plugin: attachment
22+command: udev_resource
23+_description: Attaches the output of udev_resource, for debugging purposes
24+
25 name: gcov_attachment
26 plugin: attachment
27 requires: package.name == 'lcov'

Subscribers

People subscribed via source and target branches