Change in vdsm[ovirt-4.1]: storage: No need to take locks on local storage

[Code Review]

From Yaniv Bronhaim <ybronhei(a)redhat.com&gt;:

Yaniv Bronhaim has submitted this change and it was merged.

Change subject: storage: No need to take locks on local storage
......................................................................


storage: No need to take locks on local storage

In some of the Job classes, volume leases are taken even in
case of local storage.

In this patch, volume leases will not be locked in case that the
storage domain does not support them.

Change-Id: I16e745fcdecee4ae1fad1f6deb0c2f2b869581e9
Bug-Url: https://bugzilla.redhat.com/1434304
Signed-off-by: Fred Rolland <frolland(a)redhat.com&gt;
---
M vdsm/storage/merge.py
M vdsm/storage/sdm/api/copy_data.py
M vdsm/storage/sdm/volume_info.py
3 files changed, 18 insertions(+), 14 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Yaniv Bronhaim: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, but someone else must approve
  Freddy Rolland: Verified



-- 
To view, visit https://gerrit.ovirt.org/74366
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I16e745fcdecee4ae1fad1f6deb0c2f2b869581e9
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.1
Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com&gt;
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com&gt;
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com&gt;
Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com&gt;
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com&gt;
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;