Merge lp:~brendan-donegan/checkbox/bug1224854_redux into lp:checkbox
Proposed by
Brendan Donegan
Status: | Merged |
---|---|
Approved by: | Daniel Manrique |
Approved revision: | 2399 |
Merged at revision: | 2402 |
Proposed branch: | lp:~brendan-donegan/checkbox/bug1224854_redux |
Merge into: | lp:checkbox |
Diff against target: |
46 lines (+7/-7) 2 files modified
checkbox-old/debian/changelog (+4/-5) checkbox-old/scripts/virtualization (+3/-2) |
To merge this branch: | bzr merge lp:~brendan-donegan/checkbox/bug1224854_redux |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel Manrique (community) | Approve | ||
Checkbox Developers | Pending | ||
Review via email:
|
To post a comment you must log in.
Another shot at changing the virtualization test script to use qemu-system-x86_64 instead of the deprecated 'kvm' command. Previously we had forgotten the '-machine accel=kvm:tcg' extra parameter (actually the /usr/bin/kvm script is not very clear about the importance of this) so the test was failing in our CI loop. This version of the fix has been tested thoroughly and is passing.
Please don't Approve this until Daniel has had a look since he's the one who tested the solution in the CI loop.