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

Reply via email to