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 <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
