From Yaniv Bronhaim <ybronhei(a)redhat.com>:
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>
---
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>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>