Adam Litke has submitted this change and it was merged.
Change subject: HostKSM: Gracefully handle ksmd pid lookup failure
......................................................................
HostKSM: Gracefully handle ksmd pid lookup failure
In case the pid of ksmd cannot be found we currently fail with a
ValueError exception. Check the pidof return code before trying to
convert the output to an int. If the pid cannot be found, then report 0
for get_ksmd_jiffies().
Change-Id: Iccd14ee44f4f7d8d9107e692c9faa27e7c487790
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1152182
Signed-off-by: Adam Litke <alitke(a)redhat.com>
Reviewed-on:
http://gerrit.ovirt.org/34378
Reviewed-by: Francesco Romani <fromani(a)redhat.com>
---
M mom/Collectors/HostKSM.py
1 file changed, 14 insertions(+), 3 deletions(-)
Approvals:
Adam Litke: Verified; Looks good to me, approved
Francesco Romani: Looks good to me, but someone else must approve
--
To view, visit
http://gerrit.ovirt.org/34378
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iccd14ee44f4f7d8d9107e692c9faa27e7c487790
Gerrit-PatchSet: 3
Gerrit-Project: mom
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org