Change in ovirt-engine[master]: core: QXL as default for VNC

ahadas at redhat.com ahadas at redhat.com
Mon Apr 11 13:50:14 UTC 2016


Arik Hadas has submitted this change and it was merged.

Change subject: core: QXL as default for VNC
......................................................................


core: QXL as default for VNC

>From 4.0, the default VNC driver is QXL (cirrus for a <v4.0 cluster).

With this patch, the osinfo-defaults is updated for 4.0 leading to:
- displaying VNC / [QXL|VGA] in Edit VM dialog
- QXL as a default for VNC in Add Vm REST action

The VNC driver is automatically updated from Cirrus to QXL on:
- cluster upgrade (i.e. from 3.6 to 4.0, UpdateClusterCommand) by calling
     UpdateVmCommand for all cluster VMs
- import from OVF originating in previous cluster version

There's no change for cluster compatibility version < 4.0.

Change-Id: I5a98e38476d855a1a684750d7af10f9159618280
Bug-Url: https://bugzilla.redhat.com/1302657
Signed-off-by: Marek Libra <mlibra at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateClusterCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportUtils.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommandBase.java
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java
M packaging/conf/osinfo-defaults.properties
6 files changed, 73 insertions(+), 7 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Shmuel Leib Melamud: Looks good to me, but someone else must approve
  Marek Libra: Verified
  Arik Hadas: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/55636
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5a98e38476d855a1a684750d7af10f9159618280
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Marek Libra <mlibra at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marek Libra <mlibra at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzamazal at redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Yaniv Kaul <ykaul at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list