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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Shu Ming <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: humble devassy <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
