Saggi Mizrahi has posted comments on this change.
Change subject: Refactor communication infra
......................................................................
Patch Set 15: (2 inline comments)
....................................................
File vdsm_api/BindingJsonRpc.py
Line 37: self._reactors = reactors
Line 38:
Line 39: def _createTcpReactor(self, cfg):
Line 40: address = cfg.get("ip", "0.0.0.0")
Line 41: port = cfg.get("port", 4044)
I remember already replying on this comment. Anyway, it doesn't matter as VDSM
always sends the port number from it's own defaults.
If I didn't put a default here I would have had to put error handling code here
and in every other place in the stack and that would have just been annoying.
Line 42: return TCPReactor((address, port), self.server)
Line 43:
Line 44: def start(self):
Line 45: for reactor in self._reactors:
....................................................
File vdsm_api/jsonrpc/tcpReactor.py
Line 63: buffLen = len(self._buffer)
Line 64: if buffLen < _Size.size:
Line 65: return None
Line 66:
Line 67: if buffLen > self.buffsize:
It is a remnant of days gone by. I'll remove it in another patch, please don't
make me rebase everything and loos all the +1s
Line 68: self._conn.close()
Line 69: self._sendData = []
Line 70:
Line 71: msgLen = _Size.unpack(self._buffer[:_Size.size])[0]
--
To view, visit http://gerrit.ovirt.org/9986
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I7a65364824cc8433263ccf6ed44f1f97141eff2c
Gerrit-PatchSet: 15
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
