Change in vdsm[ovirt-3.6.3]: hooks: make sure to handle vdsm exception in the bridge

fromani at redhat.com fromani at redhat.com
Thu Feb 18 12:37:31 UTC 2016


Francesco Romani has submitted this change and it was merged.

Change subject: hooks: make sure to handle vdsm exception in the bridge
......................................................................


hooks: make sure to handle vdsm exception in the bridge

We need to make sure to handle hook errors properly and pass them to
the client.


Change-Id: Ic9c0e953f96827737eede22ab39a1f7689c30944
Signed-off-by: pkliczewski <piotr.kliczewski at gmail.com>
Bug-Url: https://bugzilla.redhat.com/1295778
Reviewed-on: https://gerrit.ovirt.org/52984
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/53696
Continuous-Integration: Francesco Romani <fromani at redhat.com>
---
M tests/bridgeTests.py
M vdsm/rpc/Bridge.py
2 files changed, 19 insertions(+), 0 deletions(-)

Approvals:
  Piotr Kliczewski: Verified
  Francesco Romani: Looks good to me, approved; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/53696
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic9c0e953f96827737eede22ab39a1f7689c30944
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6.3
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list