Dan Kenigsberg has submitted this change and it was merged.
Change subject: virt: graphdev: add the GraphicsDevice class
......................................................................
virt: graphdev: add the GraphicsDevice class
this patch introduces the Graphics Device class to be used
to define the Vm display properties.
No more than one Graphics Device is supported.
If VDSM receives a Graphic Device definition, it uses that;
otherwise, tries to fallback to display* parameters received as input.
VDSM will continue to supply the legacy display* parameters
in stats output and in the conf dictionary (VmDefinition)
to preserve the backward compatibility. This includes migrations,
so a VM created with a single new style graphic device can be
migrated to old VDSMs without breaking cluster compatibility.
Change-Id: I82e4414aa3efe7c756ba8ef9b7b47e591613a717
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
Reviewed-on:
http://gerrit.ovirt.org/26895
Reviewed-by: Antoni Segura Puimedon <asegurap(a)redhat.com>
Reviewed-by: Dan Kenigsberg <danken(a)redhat.com>
---
M tests/functional/virtTests.py
M tests/vmTests.py
M tests/vmTestsData.py
M vdsm/API.py
M vdsm/virt/vm.py
M vdsm_api/vdsmapi-schema.json
6 files changed, 393 insertions(+), 132 deletions(-)
Approvals:
Antoni Segura Puimedon: Looks good to me, but someone else must approve
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Verified
--
To view, visit
http://gerrit.ovirt.org/26895
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I82e4414aa3efe7c756ba8ef9b7b47e591613a717
Gerrit-PatchSet: 29
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpoledni(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server