Change in vdsm[master]: always teardown snapshot's memory volume

danken at redhat.com danken at redhat.com
Thu May 22 14:30:42 UTC 2014


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 <ahadas at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/26544
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-by: Francesco Romani <fromani at 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 <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list