Checkbox

Merge lp:~pieq/checkbox/fix-1621815-multi-head-job into lp:checkbox

  1. fix-1621815-multi-head-job
  2. Merge into trunk
Proposed by Pierre Equoy
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 4494
Merged at revision: 4494
Proposed branch: lp:~pieq/checkbox/fix-1621815-multi-head-job
Merge into: lp:checkbox
Diff against target: 12 lines (+1/-1)
1 file modified
providers/plainbox-provider-checkbox/jobs/monitor.txt.in (+1/-1)
To merge this branch: bzr merge lp:~pieq/checkbox/fix-1621815-multi-head-job
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+305553@code.launchpad.net

Description of the change

Tested using the faulty device:

When running the multi-head job prior to applying the patch, the job is skipped.
When running the job after applying the patch, it can be tested properly.

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

Form factors and dmi...

+1

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/monitor.txt.in'
2--- providers/plainbox-provider-checkbox/jobs/monitor.txt.in 2016-06-07 02:32:19 +0000
3+++ providers/plainbox-provider-checkbox/jobs/monitor.txt.in 2016-09-13 04:38:01 +0000
4@@ -85,7 +85,7 @@
5 unit: template
6 template-resource: graphics_card
7 id: monitor/{index}_multi-head_{product_slug}
8-requires: dmi.product in ['Desktop','Low Profile Desktop','Tower','Mini Tower']
9+requires: dmi.product in ['Desktop','Low Profile Desktop','Tower','Mini Tower', 'Space-saving']
10 plugin: manual
11 category_id: 2013.com.canonical.plainbox::monitor
12 _purpose:

Subscribers

People subscribed via source and target branches