Francesco Romani has posted comments on this change.

Change subject: virt: graphdev: support multiple graphics devices
......................................................................


Patch Set 17: Verified+1

Vetrified with http://gerrit.ovirt.org/#/c/26895/ , 
http://gerrit.ovirt.org/#/c/27595/ and http://gerrit.ovirt.org/#/c/27596/

Verification (all the following using unpatched engine 3.5.0 snapshot:
* creation flow: booted a VM, connected with spice
* recovery: booted a VM, restarted VDSM, connected with spice
* migration: migrated back and forth another host running VDSM 4.14.8.1
* hibernate:  booted a VM, restarted and connected with spice.
* verified the output of vdsClient
* cursory look at vdsm logs and to engine logs seems OK.

Found
Received a spice Device without an address when processing VM 
025d59e3-f12c-41b8-be3f-de8dc22d23ce devices, skipping device: {port=5900, 
specParams={keyMap=en-us, copyPasteEnable=false, displayIp=0, 
fileTransferEnable=false}, device=spice, type=graphics}

but that is expected since the graphics device do not have PCI addresses.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5be348b342359d42c878937dca27454fe206a35a
Gerrit-PatchSet: 17
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to