Merge lp:~roadmr/checkbox/cdts-fixes into lp:checkbox

Proposed by Daniel Manrique
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 2983
Merged at revision: 2982
Proposed branch: lp:~roadmr/checkbox/cdts-fixes
Merge into: lp:checkbox
Diff against target: 20 lines (+2/-1)
1 file modified
providers/plainbox-provider-checkbox/jobs/info.txt.in (+2/-1)
To merge this branch: bzr merge lp:~roadmr/checkbox/cdts-fixes
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+218669@code.launchpad.net

Commit message

5164114 providers:checkbox: acpi_sleep_attachment should succed and report even if acpi/sleep doesn't exist (LP: #1316695)
37a55b3 providers:checkbox: installer_debug.gz job should run as root (LP: #1316696)

Description of the change

Fixes two small job bugs:

5164114 providers:checkbox: acpi_sleep_attachment should succed and report even if acpi/sleep doesn't exist (LP: #1316695)
37a55b3 providers:checkbox: installer_debug.gz job should run as root (LP: #1316696)

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'providers/plainbox-provider-checkbox/jobs/info.txt.in'
--- providers/plainbox-provider-checkbox/jobs/info.txt.in 2014-04-07 16:33:10 +0000
+++ providers/plainbox-provider-checkbox/jobs/info.txt.in 2014-05-07 17:29:02 +0000
@@ -126,7 +126,7 @@
126126
127plugin: attachment127plugin: attachment
128id: acpi_sleep_attachment128id: acpi_sleep_attachment
129command: [ -e /proc/acpi/sleep ] && cat /proc/acpi/sleep129command: [ -e /proc/acpi/sleep ] && cat /proc/acpi/sleep || echo "No /proc/acpi/sleep found"
130estimated_duration: 0.5130estimated_duration: 0.5
131_description: Attaches the contents of /proc/acpi/sleep if it exists.131_description: Attaches the contents of /proc/acpi/sleep if it exists.
132132
@@ -180,6 +180,7 @@
180180
181plugin: attachment181plugin: attachment
182id: installer_debug.gz182id: installer_debug.gz
183user: root
183command: [ -e /var/log/installer/debug ] && gzip -9 -c /var/log/installer/debug184command: [ -e /var/log/installer/debug ] && gzip -9 -c /var/log/installer/debug
184estimated_duration: 0.1185estimated_duration: 0.1
185_description: Attaches the installer debug log if it exists.186_description: Attaches the installer debug log if it exists.

Subscribers

People subscribed via source and target branches