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 <ali...@redhat.com>
Reviewed-on: http://gerrit.ovirt.org/34378
Reviewed-by: Francesco Romani <from...@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 <ali...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Martin Sivák <msi...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to