Michal Skrivanek has posted comments on this change. Change subject: vdsm: Adding guest agent API versioning support ......................................................................
Patch Set 1: Looks good to me, but someone else must approve (1 inline comment) .................................................... File vdsm/guestIF.py Line 98: MAX_MESSAGE_SIZE = 2 ** 20 # 1 MiB for now Line 99: Line 100: def __init__(self, socketName, channelListener, log, user='Unknown', Line 101: ips='', connect=True): Line 102: self.apiVersion = 0 vdsm proposes the highest it supports, guest-agent replies with the lower value of the highest it supports or vdsm. Both sides support at least version 0. Line 103: self.log = log Line 104: self._socketName = socketName Line 105: self._stopped = True Line 106: self.guestStatus = None -- To view, visit http://gerrit.ovirt.org/17004 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9095b528c2c910f12d5f170088a458bf11c71910 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Martin Sivák <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Peter V. Saveliev <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
