Change in vdsm[master]: virt: graphdev: handle ticket for multiple devices

fkobzik at redhat.com fkobzik at redhat.com
Wed Apr 23 09:29:07 UTC 2014


Frank Kobzik has posted comments on this change.

Change subject: virt: graphdev: handle ticket for multiple devices
......................................................................


Patch Set 7:

(1 comment)

http://gerrit.ovirt.org/#/c/26897/7/vdsm/virt/vm.py
File vdsm/virt/vm.py:

Line 4256:         return {'status': doneCode, 'vmList': self.status()}
Line 4257: 
Line 4258:     def setTicket(self, otp, seconds, connAct, params):
Line 4259:         res = {'status': doneCode}
Line 4260:         for graphics in _domParseStr(self._dom.XMLDesc(0)).childNodes[0]. \
I'm not sure if setting ticket for both devices is ok, but whatever. (it could save us from some bc issues).
Line 4261:                 getElementsByTagName('graphics'):
Line 4262:             res = self._setTicketForGraphicDev(graphics, otp,
Line 4263:                                                seconds, connAct, params)
Line 4264:             if res['status']['code'] != 0:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I62f56ea47399b0d26aafce053ea1201457564b73
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Frank Kobzik <fkobzik at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpoledni at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list