Change in vdsm[master]: supervdsm: log actual error in ProxyCaller

Yaniv Bronhaim has posted comments on this change.

Change subject: supervdsm: log actual error in ProxyCaller
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

http://gerrit.ovirt.org/#/c/35475/1/vdsm/supervdsm.py
File vdsm/supervdsm.py:

Line 53:             self._supervdsmProxy._connect()
Line 54:             self.log.exception("Could not call method")
Line 55:             raise RuntimeError(
Line 56:                 "Broken communication with supervdsm. Failed call to
%s"
Line 57:                 % self._funcName)
you should see this exception and this should be enough... I don't understand the
reason for that patch
Line 58: 
Line 59: 
Line 60: class SuperVdsmProxy(object):
Line 61:     """


-- 
To view, visit http://gerrit.ovirt.org/35475
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id8a1c83d2a963002d3c793398bb62679b77f5f47
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes