Change in ovirt-engine[master]: core: fix run once starts with wrong display type

ofrenkel at redhat.com ofrenkel at redhat.com
Thu Jan 3 07:56:09 UTC 2013


Omer Frenkel has submitted this change and it was merged.

Change subject: core: fix run once starts with wrong display type
......................................................................


core: fix run once starts with wrong display type

This patch fix the following bug: trying to run once a VM which has vnc
console set in its default settings, with spice console results in starting
qemu process with '-vga cirrus' instead of '-vga qxl'.

There was a special treatment for the case of run once with vnc console
a vm that has spice console in its default settings, but no treatment
for the scenario described above.

The fix is to add qxl video device in that case.

Change-Id: I635307847edf764b174230203ee355987b948ce4
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java
1 file changed, 40 insertions(+), 28 deletions(-)

Approvals:
  Omer Frenkel: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/10537
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I635307847edf764b174230203ee355987b948ce4
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>



More information about the Engine-commits mailing list