Eduardo has posted comments on this change. Change subject: Use a shared resource for images ......................................................................
Patch Set 3: I would prefer that you didn't submit this (1 inline comment) The actual semantics (before the patch) is buggy and dangerous. Sorry. IMHO: Since the image can't be changed on the HSM and the local scope Image lock, tacking this lock on HSM is irrelevant. The rw volume permission should be changed only when adding/removing a leaf of a volume, and only on SPM. When running a VM all the chain of volumes should be activated during prepare. Only this is required and it is a local (host) operation. The scope of the rw permission is the cluster and should be changed on the SPM only and is not related to running VM's as said before. .................................................... File vdsm/storage/resourceFactories.py Line 64: lvm.setrwLV(self._vg, self._lv, True) This code should not be called for an HSM. If the lock can be shared because this is not the SPM, then this is a cluster violation. -- To view, visit http://gerrit.ovirt.org/429 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8126a7ef9758b3625f4957997ffbe4154e93cc97 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Igor Lvovsky <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
