Nova boot on PowerKVM fails when video type is set to 'cirrus'

[Lance Bragstad]

When deploying to a compute node hosted on a PowerKVM system, nova boots fail to spawn when video type is set to 'cirrus' in the XML definition. This is the default value set and was a part of this blueprint:

https://blueprints.launchpad.net/nova/+spec/libvirt-video-driver-selection

and this change:

https://review.openstack.org/#/c/58614/8

When trying to launch an instance, the following can be observed on the PowerKVM compute node: http://paste.openstack.org/show/62144/

and the XML definition here: http://paste.openstack.org/show/62145/

One possible solution for this would be to detect the architecture of the compute node and to not default video type to 'cirrus', until PowerKVM supports cirrus emulation. This is already partially done by https://review.openstack.org/#/c/66992/10 and can be used in https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L3211-L3225 without causing issues with the current implementation, as discussed in #openstack-nova with mriedem, dansmith, and russellb.

[Matt Riedemann (mriedem)]

Patch is here: https://review.openstack.org/#/c/70046/

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/70046
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=852c7f39692473ece176c5ef2cf410b21d774dad
Submitter: Jenkins
Branch: master

commit 852c7f39692473ece176c5ef2cf410b21d774dad
Author: Lance Bragstad <email address hidden>
Date: Thu Jan 30 04:01:10 2014 +0000

    Default video type to 'vga' for PowerKVM

    PowerKVM currently doesn't support 'cirrus' video type. For now, we can
    detect if Nova compute is running on ppc64 or ppc architecture systems
    and set the default to 'vga', which is supported on PowerKVM.

    This patch did some reordering of the logic in
    nova.virt.libvirt.driver.LibvirtDriver.get_guest_config in an attempt to
    set the video.type depending on the compute host architecture and
    implementation specifics. The video.type can still be set using
    image_meta.

    Change-Id: I49a1a8e6d9ebdc28d876d2f958c8cb960892f322
    Closes-Bug: 1274367