Allon Mureinik has posted comments on this change.
Change subject: BZ#826022 Disable the sanlock volume leases ......................................................................
Patch Set 1: (1 inline comment)
.................................................... File vdsm/libvirtvm.py Line 1193: Is this an issue with SANLock or with VDSM's interaction/usage of SANLock?
I'm not a fan of leaving commented out blocks of code:
If this is a SANlock issue, perhaps we should have a configuration value of which version of SANLock supports this (with the value set to 999 or something ridiculous like that)?
-- To view, visit http://gerrit.ovirt.org/4872 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: Ib799ce25cb4c8a363d4021d8a1c63c7925d33855 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Allon Mureinik amureini@redhat.com Gerrit-Reviewer: Eduardo ewarszaw@redhat.com