Change in vdsm[master]: Remove validateAttachedDomain from detachSD

abaron at redhat.com abaron at redhat.com
Mon Feb 20 13:04:24 UTC 2012


Ayal Baron has posted comments on this change.

Change subject: Remove validateAttachedDomain from detachSD
......................................................................


Patch Set 2:

I've discussed this with Edu and he managed to convince me that we should not remove this validation (it would only be relevant in case detach is normally issued and fails between removal from the master storage domain and the sd backref), otherwise there is no pool to perform the action.

--
To view, visit http://gerrit.ovirt.org/2074
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9bbf19973cafd869f0e0ce3dfdf38b3619caa110
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>


More information about the vdsm-patches mailing list