Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsm: Add additional exception handling within vmChannels.py ......................................................................
vdsm: Add additional exception handling within vmChannels.py At present any exception thrown by _epoll.poll(1) is not caught. This patch adds additional exception handling to catch and log any exceptions thrown by the thread. Change-Id: Ie733ca7ccd2689f6041ee18bbe335f355a89ac55 Bug-Url: https://bugzilla.redhat.com/918541 Signed-off-by: Lee Yarwood <[email protected]> Signed-off-by: Vinzenz Feenstra <[email protected]> --- M vdsm/vmChannels.py 1 file changed, 9 insertions(+), 3 deletions(-) Approvals: Federico Simoncelli: Looks good to me, but someone else must approve Jason Dillaman: Verified; Looks good to me, but someone else must approve Gal Hammer: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Peter V. Saveliev: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/13093 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie733ca7ccd2689f6041ee18bbe335f355a89ac55 Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Lee Yarwood <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Gal Hammer <[email protected]> Gerrit-Reviewer: Jason Dillaman <[email protected]> Gerrit-Reviewer: Lee Yarwood <[email protected]> Gerrit-Reviewer: Peter V. Saveliev <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
