Checkbox Provider - Base

Merge ~leftyfb/plainbox-provider-checkbox:1659929-fix-virt-cpu into plainbox-provider-checkbox:master

  1. Git
  2. lp:~leftyfb/plainbox-provider-checkbox
  3. 1659929-fix-virt-cpu
  4. Merge into master
Proposed by Mike Rushton
Status: Merged
Approved by: Jeff Lane 
Approved revision: 3cffaa7ed9942476651c56557ef251bebde7c990
Merged at revision: b5d5b5baa998bd8252689e18fa653fa01bd86518
Proposed branch: ~leftyfb/plainbox-provider-checkbox:1659929-fix-virt-cpu
Merge into: plainbox-provider-checkbox:master
Diff against target: 13 lines (+1/-1)
1 file modified
bin/virtualization (+1/-1)
Reviewer Review Type Date Requested Status
Jeff Lane  Approve
Review via email: mp+315823@code.launchpad.net

Description of the change

Fix for bug #1659929

Replaced hardcoded cpu type entry in virtualization script from 'POWER8' to 'host' to match the cpu type of the host.

To post a comment you must log in.
Revision history for this message
Jeff Lane  (bladernr) wrote :

Tested on standard Power8 an Power8NVL.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/bin/virtualization b/bin/virtualization
2index 748c60c..0ee74e9 100755
3--- a/bin/virtualization
4+++ b/bin/virtualization
5@@ -112,7 +112,7 @@ QEMU_ARCH_CONFIG = {
6 'qemu_extra_args': [
7 '-enable-kvm',
8 '-machine', 'pseries,usb=off',
9- '-cpu', 'POWER8',
10+ '-cpu', 'host',
11 ],
12 },
13 's390x': {

Subscribers

People subscribed via source and target branches