illegal video driver for PPC64 little endian system

[jinweixiao]

for openstack kilo version, when creating a instance, libvirt creates cirrus video type in template xml file while is not supported for PPC64 little endian system. I debug the code and finally find mistake in function _add_video_driver of file nova/virt/libvirt/driver.py
In function, there is a logic that video driver is determinded by guest arch. If arch is in (PPC, PPC64) then return vga, otherwise video driver is determined by other options. For PPC64 little endian system, guest arch is PPC64LE so that video driver is determined by other option (In our environment, with kvm virt type and spice disabled , video driver is determined by hw_video_model) which makes video driver is cirrus . Exception happens when creating vm instance because cirrus video driver is not supported on power hardware.
I add PPC64LE arch in the guestarch option and it does works. The patch will be attached.

[Sudipta Biswas (sbiswas7)]

@xiaojinwei001 : You seem to have posted a patch but this is not the way - we usually fix bugs in Nova.
Would you want me to push these changes? You could do so too, by assigning the bug to yourself and then creating a gerrit review of your patch. Let me know.

[Sudipta Biswas (sbiswas7)]

Removing myself as the assigned. Util the above comment is answered.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/279227

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/279227
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=87069e7bf788ec4e80bd340bcb97d57117cbc4d2
Submitter: Jenkins
Branch: master

commit 87069e7bf788ec4e80bd340bcb97d57117cbc4d2
Author: Sean Dague <email address hidden>
Date: Thu Feb 11 13:19:57 2016 -0500

    Fix reported ppc64le bug on video selection

    ppc64le apparently is the same as other ppc plaforms in it's video
    selection. This one line fix was put into a reported bug and addresses
    this for people.

    Co-Authored-By: [e-mail]

    Change-Id: I44283f19823bf39159633fa93f575e306bcf1970
    Closes-Bug: #1523742

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/nova 13.0.0.0b3 development milestone.