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(a)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(a)redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>