Nir Soffer has submitted this change and it was merged.

Change subject: virt: Add support for vgamem attribute
......................................................................


virt: Add support for vgamem attribute

SPICE/QXL video RAM sizes are currently specified in <video> domain XML
element using the following attributes: ram, vram, vgamem.  VDSM is
missing support for vgamem, this patch adds it.

Change-Id: Ic595761ef7195ec12830dd7f057471512b5c6355
Bug-Url: https://bugzilla.redhat.com/1275539
Signed-off-by: Milan Zamazal <mzama...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/50091
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <from...@redhat.com>
Reviewed-by: Nir Soffer <nsof...@redhat.com>
Reviewed-by: Vinzenz Feenstra <vfeen...@redhat.com>
Tested-by: Nir Soffer <nsof...@redhat.com>
---
M lib/api/vdsmapi-schema.json
M tests/deviceTests.py
M vdsm/virt/vmdevices/core.py
3 files changed, 12 insertions(+), 7 deletions(-)

Approvals:
  Nir Soffer: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests
  Vinzenz Feenstra: Looks good to me, but someone else must approve
  Francesco Romani: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic595761ef7195ec12830dd7f057471512b5c6355
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Pavel Zhukov <pzhu...@redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to