Change in vdsm[master]: Related to BZ#713173 - Add killProc rollback to deleteImage

danken at redhat.com danken at redhat.com
Sun Jul 17 11:16:14 UTC 2011


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Related to BZ#713173 - Add killProc rollback to deleteImage
......................................................................


Related to BZ#713173 - Add killProc rollback to deleteImage

When we run deleteImage with post-zeroing flag, we need to issue 'dd' for this.
The rollback procedure for deleteImage it's actually roll forward.
In this situation we need first to kill previous 'dd' operation that may hold
file handler (or LV) open and prevent us to run the new deleteImage.
Note that in case we need to run this roll forward action on different host (SPM switchover)
the 'dd' killing will do nothing and still may fail.

Change-Id: I1407626ab06583f5507f3aaa8ebf2a4e4edd60f9
---
M vdsm/storage/blockVolume.py
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved


--
To view, visit http://gerrit.usersys.redhat.com/707
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1407626ab06583f5507f3aaa8ebf2a4e4edd60f9
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: David Naori <dnaori at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list