Merge lp:~bladernr/checkbox/change-virt-test-ram into lp:checkbox

Proposed by Jeff Lane 
Status: Merged
Approved by: Po-Hsu Lin
Approved revision: 4317
Merged at revision: 4317
Proposed branch: lp:~bladernr/checkbox/change-virt-test-ram
Merge into: lp:checkbox
Diff against target: 12 lines (+1/-1)
1 file modified
providers/plainbox-provider-checkbox/bin/virtualization (+1/-1)
To merge this branch: bzr merge lp:~bladernr/checkbox/change-virt-test-ram
Reviewer Review Type Date Requested Status
Po-Hsu Lin Approve
Review via email: mp+292434@code.launchpad.net

Description of the change

changes the virtualization script so that it creates a VM with 1GB RAM instead of 256MB. Recent kernels need more than 256MB to boot and because of this limit the tests are failing on non-x86 architectures.

To post a comment you must log in.
Revision history for this message
Po-Hsu Lin (cypressyew) wrote :

Looking good, thanks.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'providers/plainbox-provider-checkbox/bin/virtualization'
--- providers/plainbox-provider-checkbox/bin/virtualization 2016-04-13 21:28:51 +0000
+++ providers/plainbox-provider-checkbox/bin/virtualization 2016-04-20 18:55:17 +0000
@@ -126,7 +126,7 @@
126 # Parameters common to all architectures126 # Parameters common to all architectures
127 self.params = [127 self.params = [
128 self.config['qemu_bin'],128 self.config['qemu_bin'],
129 "-m", "256",129 "-m", "1024",
130 "-display", "none",130 "-display", "none",
131 "-nographic",131 "-nographic",
132 "-net", "nic",132 "-net", "nic",

Subscribers

People subscribed via source and target branches