Change in vdsm[master]: Storage: Check that underlying file system supports direct I...

danken at redhat.com danken at redhat.com
Mon Feb 4 18:48:37 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Storage: Check that underlying file system supports direct IO
......................................................................


Storage: Check that underlying file system supports direct IO

- Check when creating a domain
- Check when creating a domain object

This is done so that we don't allow posixfs domains on top of targets
that don't support direct IO.

Direct IO is needed in order to read the metadata reliably in a
clustered environment. It is also a required feature for Sanlock.

Change-Id: Icf14d1c4737a88e693e5bebb896aef382b8b424c
Signed-off-by: Saggi Mizrahi <smizrahi at redhat.com>
---
M vdsm/storage/fileSD.py
M vdsm/storage/localFsSD.py
M vdsm/storage/nfsSD.py
M vdsm/storage/remoteFileHandler.py
M vdsm/storage/storage_exception.py
5 files changed, 38 insertions(+), 1 deletion(-)

Approvals:
  Ayal Baron: Looks good to me, approved
  Saggi Mizrahi: Verified
  Federico Simoncelli: Looks good to me, but someone else must approve
  Dan Kenigsberg: 


--
To view, visit http://gerrit.ovirt.org/9749
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icf14d1c4737a88e693e5bebb896aef382b8b424c
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Shu Ming <shuming at linux.vnet.ibm.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: humble devassy <hchiramm at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list