Change in vdsm[master]: vdsm-tool: service operations do not trigger abrt warning on...
zhshzhou at linux.vnet.ibm.com
zhshzhou at linux.vnet.ibm.com
Wed May 29 04:07:20 UTC 2013
Zhou Zheng Sheng has posted comments on this change.
Change subject: vdsm-tool: service operations do not trigger abrt warning on RHEL6
......................................................................
Patch Set 1: (1 inline comment)
....................................................
Commit Message
Line 8:
Line 9: When an service management operation fails, the service module raises a
Line 10: RuntimeError, this triggers abrt warning of un-caught Python exceptions.
Line 11: This patch prints the error message to standard error and returns 1
Line 12: instead of raising exception.
In another 2 patches I add the ability to distinguish a down service from a non-existing service. This patch is to suppress un-caught exception on downed service when we probe the status. The current patch set suppress all exceptions, not good. I'm writing a new one to just suppressing exception for service-status.
Line 13:
Line 14: Change-Id: Iddac6c6d1b6f47fe653ca4a86cc3a89f5ed4efb6
--
To view, visit http://gerrit.ovirt.org/15098
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iddac6c6d1b6f47fe653ca4a86cc3a89f5ed4efb6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: Aravinda VK <avishwan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list