Francesco Romani has posted comments on this change. Change subject: vm: graphics: fix console open after restore ......................................................................
Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/44842/1/vdsm/virt/vmdevices/graphics.py File vdsm/virt/vmdevices/graphics.py: Line 137: if not utils.tobool(self.specParams.get('disableTicketing', False)): Line 138: attrs['passwd'] = '*****' Line 139: attrs['passwdValidTo'] = '1970-01-01T00:00:01' Line 140: Line 141: def updateXML(self, domXML): obviously I'm not proud nor happy about this bruteforce, but proper fix needs a new architecture for devices, and we even not started discussing that. Line 142: if self.device == 'spice': Line 143: domObj = etree.fromstring(domXML) Line 144: devices = domObj.findall('devices') Line 145: -- To view, visit https://gerrit.ovirt.org/44842 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic9f11e2d87cc715c77d305c005c1cd7f502d506d Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
