Vinzenz Feenstra has submitted this change and it was merged. Change subject: hooks: Make execution more robust ...................................................................... hooks: Make execution more robust Previously, exceptions raised due to executing potentially not executable or inaccessible hooks caused exceptions to be passed through to the VirtIO read loop. Which caused that any other hook configured after that not being executed. This patch handles the OSErrors potentially raised by the subprocess usage Change-Id: I951f6e6ecabdebb9abeba7449329a363296411f7 Bug-Url: https://bugzilla.redhat.com/1343017 Signed-off-by: Vinzenz Feenstra <vfeenstr@redhat.com> (cherry picked from commit 22d972a1c4b5ea0c17992f6ce4e7cae87b8e24e4) --- M ovirt-guest-agent/hooks.py 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/58799 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I951f6e6ecabdebb9abeba7449329a363296411f7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Vinzenz Feenstra <vfeenstr@redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>