Change in vdsm[master]: Related to BZ#726400 do not remove links to known domains

danken at redhat.com danken at redhat.com
Mon Sep 19 10:44:15 UTC 2011


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Related to BZ#726400 do not remove links to known domains
......................................................................


Related to BZ#726400 do not remove links to known domains

Code introduced in commit 8ca7b3998f63d7555d253a66e02a23d577652233 was
buggy as it kept oldLinks intact after identifying a domain dir within
it. Later, all oldLinks were removed since
(oldie not in domUUIDs) is always True.

This patch reverts to former logic: as a domaindir is identified, it is
pruned from oldLinks. Left oldLinks are removed from disk.

Change-Id: I8bd8622afa1c15a249ef5e6356604b6db2af94ce
---
M vdsm/storage/sp.py
1 file changed, 9 insertions(+), 8 deletions(-)

Approvals:
  Dan Kenigsberg: Verified
  Ayal Baron: Looks good to me, but someone else must approve
  Saggi Mizrahi: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8bd8622afa1c15a249ef5e6356604b6db2af94ce
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo Warszawski <ewarszaw at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list