Nir Soffer has posted comments on this change.

Change subject: fileUtils: Make memory filesystems support configurable
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/27236/1//COMMIT_MSG
Commit Message:

Line 28:     memory_filesystems_enable = false
Line 29: 
Line 30: Note: When memory_filesystems_enable is disabled, some vdsm tests that
Line 31: try to use direct io on /tmp will fail, this is expected until these
Line 32: tests are fixed to use filesystem that supports direct io.
> Alternatively, pathRequiresFlagForDirectIO can be defined as "return True" 
I agree - but I would like to remove this check and use real filesystem for 
testing direct io instead.
Line 33: 
Line 34: Change-Id: Id539d7f610b833f4a695dc944d3663c29331286e
Line 35: Relates-to: http://gerrit.ovirt.org/9595
Line 36: Relates-to: http://gerrit.ovirt.org/9661


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id539d7f610b833f4a695dc944d3663c29331286e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to