Checkbox

Merge lp:~ara/checkbox/fix_1470758 into lp:checkbox

  1. fix_1470758
  2. Merge into trunk
Proposed by Ara Pulido
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 3871
Merged at revision: 3871
Proposed branch: lp:~ara/checkbox/fix_1470758
Merge into: lp:checkbox
Diff against target: 19 lines (+2/-0)
1 file modified
providers/plainbox-provider-checkbox/jobs/suspend.txt.in (+2/-0)
To merge this branch: bzr merge lp:~ara/checkbox/fix_1470758
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+263623@code.launchpad.net

Description of the change

Added missing AC requirements to the after suspend AC tests (auto) that is causing some false positives in the mini CI loop

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

Thanks, looks good, +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/suspend.txt.in'
2--- providers/plainbox-provider-checkbox/jobs/suspend.txt.in 2015-06-02 08:22:55 +0000
3+++ providers/plainbox-provider-checkbox/jobs/suspend.txt.in 2015-07-02 08:30:51 +0000
4@@ -807,6 +807,7 @@
5 requires:
6 device.category == 'WIRELESS'
7 environment.ROUTERS == 'multiple'
8+ IEEE_80211.ac == 'supported'
9 user: root
10 environ: WPA_AC_SSID WPA_AC_PSK
11 command:
12@@ -829,6 +830,7 @@
13 requires:
14 device.category == 'WIRELESS'
15 environment.ROUTERS == 'multiple'
16+ IEEE_80211.ac == 'supported'
17 user: root
18 environ: OPEN_AC_SSID
19 command:

Subscribers

People subscribed via source and target branches