Change in vdsm[master]: localFsSD: Use the right manifest.

nsoffer at redhat.com nsoffer at redhat.com
Thu Feb 18 18:15:06 UTC 2016


Nir Soffer has submitted this change and it was merged.

Change subject: localFsSD: Use the right manifest.
......................................................................


localFsSD: Use the right manifest.

Local storage domain should use its own manifest instead of the
inherited one from FileStorageDomain.

FileStorageDomain's manifest directs the domain creation to lock
the storage using sanlock, which is used for shared storage locking.

As local storage domain is used only by its containing host, there is 
no point to use sanlock. instead we use a local lock.

Change-Id: I482511290032cad3813e5c8b85b59d2369da2e90
Bug-Url:https://bugzilla.redhat.com/1305768
Signed-off-by: Amit Aviram <aaviram at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/53688
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Vered Volansky <vvolansk at redhat.com>
Reviewed-by: Freddy Rolland <frolland at redhat.com>
Reviewed-by: Allon Mureinik <amureini at redhat.com>
Reviewed-by: Adam Litke <alitke at redhat.com>
---
M vdsm/storage/localFsSD.py
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Adam Litke: Looks good to me, approved
  Nir Soffer: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Vered Volansky: Looks good to me, but someone else must approve
  Allon Mureinik: Looks good to me, but someone else must approve
  Amit Aviram: Verified
  Freddy Rolland: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I482511290032cad3813e5c8b85b59d2369da2e90
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Amit Aviram <aaviram at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Amit Aviram <aaviram at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list