Dan Kenigsberg has submitted this change and it was merged. Change subject: net_func_tests: Rollback only when an error happens ......................................................................
net_func_tests: Rollback only when an error happens Up until now, net_func_tests used RollbackContext that always performs all the rolling back operations on context exit. This patch adds a flag to RollbackContext that if explicitely set to True, will only perform the rollback operations in case the context was left due to an exception. Change-Id: Iaeb9bf19f4bf63d3705cfdda92aa6ec7513a8b98 Signed-off-by: Antoni S. Puimedon <[email protected]> Reviewed-on: http://gerrit.ovirt.org/29817 Reviewed-by: Piotr Kliczewski <[email protected]> Tested-by: Ido Barkan <[email protected]> Reviewed-by: mooli tayer <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M lib/vdsm/utils.py M tests/functional/networkTests.py 2 files changed, 7 insertions(+), 2 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Ido Barkan: Verified mooli tayer: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29817 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaeb9bf19f4bf63d3705cfdda92aa6ec7513a8b98 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Ido Barkan <[email protected]> Gerrit-Reviewer: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: mooli tayer <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
