Change in vdsm[master]: BZ#733373 - Remove the pool level vsm and tasks links.

danken at redhat.com danken at redhat.com
Sun Sep 11 10:19:24 UTC 2011


Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#733373 - Remove the pool level vsm and tasks links.
......................................................................


BZ#733373 - Remove the pool level vsm and tasks links.

As verified by the previous version of this patch, the pool level
links to the tasks and vms are superflous, dangerous and can
prevent spm restart.
Since they are unused, best if they are not created at all.
cleanupSpmLinks() removed so it can't fail.
_createSpmLinks() removed too.

Change-Id: Id16baf5c23a7b71f25030c0443710f9412252367
---
M vdsm/storage/spm.py
1 file changed, 2 insertions(+), 43 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id16baf5c23a7b71f25030c0443710f9412252367
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo Warszawski <ewarszaw 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: Haim Ateya <hateya at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Yotam Oron <yoron at redhat.com>


More information about the vdsm-patches mailing list