Checkbox

Merge lp:~bladernr/checkbox/1486089-efi-only-x86 into lp:checkbox

  1. 1486089-efi-only-x86
  2. Merge into trunk
Proposed by Jeff Lane 
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 3951
Merged at revision: 3950
Proposed branch: lp:~bladernr/checkbox/1486089-efi-only-x86
Merge into: lp:checkbox
Diff against target: 12 lines (+2/-0)
1 file modified
providers/plainbox-provider-checkbox/jobs/miscellanea.txt.in (+2/-0)
To merge this branch: bzr merge lp:~bladernr/checkbox/1486089-efi-only-x86
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+268340@code.launchpad.net

Description of the change

Quick change to ensure boot-mode test only runs on x86/x86_64 architechtures where it makes sense (e.g. EFI doesn't apply to ppc64 or ARM)

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

Hey, what about aarch64 aka arm64. EFI does work and is expected to work there.

lp:~bladernr/checkbox/1486089-efi-only-x86 updated
3951. By Jeff Lane 

Include aarch64 as well as EFI works there since Trusty

Revision history for this message
Jeff Lane  (bladernr) wrote :

Ahh yeah, verified that as well. added aarch64 to the list.

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/miscellanea.txt.in'
2--- providers/plainbox-provider-checkbox/jobs/miscellanea.txt.in 2015-07-13 13:02:37 +0000
3+++ providers/plainbox-provider-checkbox/jobs/miscellanea.txt.in 2015-08-18 22:25:34 +0000
4@@ -73,6 +73,8 @@
5 plugin: shell
6 estimated_duration: 0.5
7 id: miscellanea/boot_mode
8+requires:
9+ cpuinfo.platform in ("i386", "x86_64", "aarch64")
10 _summary: EFI-mode boot test
11 _description: Succeeds when booted in EFI mode; fails otherwise
12 command:

Subscribers

People subscribed via source and target branches