Merge lp:~cypressyew/checkbox/fix-sru-suspend-adv-trigger into lp:checkbox

Proposed by Po-Hsu Lin
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 3736
Merged at revision: 3736
Proposed branch: lp:~cypressyew/checkbox/fix-sru-suspend-adv-trigger
Merge into: lp:checkbox
Diff against target: 11 lines (+0/-1)
1 file modified
providers/plainbox-provider-checkbox/jobs/suspend.txt.in (+0/-1)
To merge this branch: bzr merge lp:~cypressyew/checkbox/fix-sru-suspend-adv-trigger
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Taihsiang Ho Pending
Review via email: mp+257722@code.launchpad.net

Description of the change

Remove the dependency for the suspend attach log job, so that it won't be triggered in the SRU testing.

The possible issue is that when running N+1, user can't select single-log-check or the single-log-attach job to trigger the suspend task.

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

+1, thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'providers/plainbox-provider-checkbox/jobs/suspend.txt.in'
2--- providers/plainbox-provider-checkbox/jobs/suspend.txt.in 2015-04-23 04:20:19 +0000
3+++ providers/plainbox-provider-checkbox/jobs/suspend.txt.in 2015-04-29 07:59:12 +0000
4@@ -360,7 +360,6 @@
5
6 plugin: attachment
7 id: suspend/suspend-single-log-attach
8-depends: suspend/suspend_advanced
9 command: [ -e $PLAINBOX_SESSION_SHARE/suspend_single.log ] && cat $PLAINBOX_SESSION_SHARE/suspend_single.log
10 _description:
11 Attaches the log from the single suspend/resume test to the results

Subscribers

People subscribed via source and target branches