Change in vdsm[master]: Adding call to svdsm.ping from vdsm ping verb
smizrahi at redhat.com
smizrahi at redhat.com
Tue Aug 20 14:11:25 UTC 2013
Saggi Mizrahi has posted comments on this change.
Change subject: Adding call to svdsm.ping from vdsm ping verb
......................................................................
Patch Set 2: Code-Review-1
(1 comment)
....................................................
File vdsm/API.py
Line 1152:
Line 1153: def ping(self):
Line 1154: "Ping the server. Useful for tests"
Line 1155: try:
Line 1156: supervdsm.getProxy().ping()
Either log or return the of the original exception.
It's the thing we really want to know in those cases
Line 1157: except:
Line 1158: return {'status':
Line 1159: {'code': 1, 'message': "Couldn't ping supervdsm"}}
Line 1160:
--
To view, visit http://gerrit.ovirt.org/18329
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifc4caf642ce6d8281fa97f7f26ec39575304f339
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list