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