Dan Kenigsberg has submitted this change and it was merged. Change subject: always teardown snapshot's memory volume ......................................................................
always teardown snapshot's memory volume Teardown the memory volume of snapshot with memory as part of 'finally' block to ensure it is done in case an exception is thrown in the 'snapshot' method as well. Change-Id: If59047f7d5ba6cfa942b593683f6f9987619a7ea Signed-off-by: Arik Hadas <aha...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/26544 Reviewed-by: Dan Kenigsberg <dan...@redhat.com> Reviewed-by: Francesco Romani <from...@redhat.com> --- M vdsm/virt/vm.py 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Dan Kenigsberg: Looks good to me, approved Francesco Romani: Looks good to me, but someone else must approve Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/26544 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If59047f7d5ba6cfa942b593683f6f9987619a7ea Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches