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