Merge ~bladernr/plainbox-provider-checkbox:no-usb-s390 into plainbox-provider-checkbox:master

Proposed by Jeff Lane 
Status: Merged
Approved by: Jeff Lane 
Approved revision: d24deff8a86e0725ff64760dbea40c9a24cbfa15
Merged at revision: e8eb04725191e3ca4beaefc87a02c57022632a68
Proposed branch: ~bladernr/plainbox-provider-checkbox:no-usb-s390
Merge into: plainbox-provider-checkbox:master
Diff against target: 29 lines (+4/-0)
1 file modified
jobs/usb.txt.in (+4/-0)
Reviewer Review Type Date Requested Status
Jonathan Cave (community) Approve
Review via email: mp+327135@code.launchpad.net

Description of the change

USB jobs shouldn't run on s390x systems

To post a comment you must log in.
Revision history for this message
Jonathan Cave (jocave) wrote :

LGTM +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/jobs/usb.txt.in b/jobs/usb.txt.in
2index acca8f5..8fa63d7 100644
3--- a/jobs/usb.txt.in
4+++ b/jobs/usb.txt.in
5@@ -6,6 +6,8 @@ value-type: bool
6 plugin: shell
7 category_id: com.canonical.plainbox::usb
8 id: usb/detect
9+requires:
10+ cpuinfo.platform != 's390x'
11 estimated_duration: 1.0
12 command:
13 set -o pipefail
14@@ -227,6 +229,7 @@ estimated_duration: 45.0
15 command: removable_storage_test -l usb && removable_storage_test -s 268400000 usb
16 flags: also-after-suspend preserve-cwd
17 requires:
18+ cpuinfo.platform != 's390x'
19 package.name == 'udisks2' or snap.name == 'udisks2'
20 package.name == 'udisks2' or (snap.name == 'core' and int(snap.revision) >= 1804)
21 _summary:
22@@ -242,6 +245,7 @@ id: usb3/storage-preinserted
23 user: root
24 flags: also-after-suspend
25 requires:
26+ cpuinfo.platform != 's390x'
27 usb.usb3 == 'supported'
28 package.name == 'udisks2' or snap.name == 'udisks2'
29 package.name == 'udisks2' or (snap.name == 'core' and int(snap.revision) >= 1804)

Subscribers

People subscribed via source and target branches