Saggi Mizrahi has uploaded a new change for review. Change subject: Add more logging to JsonRpcTests to ease debugging ......................................................................
Add more logging to JsonRpcTests to ease debugging Change-Id: Iee91e8bd188423e886b4179e024102adc23d33fb Signed-off-by: Saggi Mizrahi <[email protected]> --- M tests/jsonRpcTests.py M vdsm_api/jsonrpc/__init__.py 2 files changed, 9 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/55/10255/1 diff --git a/tests/jsonRpcTests.py b/tests/jsonRpcTests.py index 376b630..40b20dd 100644 --- a/tests/jsonRpcTests.py +++ b/tests/jsonRpcTests.py @@ -24,6 +24,7 @@ from contextlib import closing import json import uuid +import logging from nose.plugins.skip import SkipTest @@ -190,6 +191,8 @@ class ProtonReactorClient(object): + log = logging.getLogger("ProtonReactorClient") + def __init__(self, brokerAddress): self._serverAddress = brokerAddress self._msngr = proton.Messenger("client-%s" % str(uuid.uuid4())) @@ -221,7 +224,11 @@ timeout *= 1000 self._msngr.timeout = timeout - self._msngr.recv(1) + self.log.debug("Waiting for message") + try: + self._msngr.recv(1) + finally: + self.log.debug("Done waiting for message") if not self._msngr.incoming: raise socket.timeout() diff --git a/vdsm_api/jsonrpc/__init__.py b/vdsm_api/jsonrpc/__init__.py index 8e58ffe..9463262 100644 --- a/vdsm_api/jsonrpc/__init__.py +++ b/vdsm_api/jsonrpc/__init__.py @@ -91,7 +91,7 @@ class JsonRpcServer(object): - log = logging.getLogger("JsonRpcServer") + log = logging.getLogger("jsonrpc.JsonRpcServer") def __init__(self, bridge): self._bridge = bridge -- To view, visit http://gerrit.ovirt.org/10255 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Iee91e8bd188423e886b4179e024102adc23d33fb Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
