Description of problem: tests/basic/fop-sanity.t fails randomly Version-Release number of selected component (if applicable): How reproducible: Randomly on the regression systems However, it is consistently reproducible if tests/basic/fop-sanity.c is modified to add the following line immediately before the call to open(): fprintf(stdout, "Test string"); Steps to Reproduce: 1. 2. 3. Actual results: Result: PASS tests/basic/fops-sanity.t .. 1..11 ok 1 ok 2 No volumes present ok 3 ok 4 ok 5 ok 6 ok 7 ok 8 volume set: success ok 9 read failed: No data available read returning junk fd based file operation 1 failed read failed: No data available read returning junk fstat failed : No data available fd based file operation 2 failed read failed: No data available read returning junk dup fd based file operation failed not ok 10 FAILED COMMAND: ./fops-sanity damn /home/nbalachandran/Projects/gluster/code/glusterfs ok 11 Failed 1/11 subtests Expected results: Test should pass Additional info:
The man page for open() states that the mode must be passed in if the flags include O_CREAT. The calls to open() in the fops-sanity.c did not include the mode argument. This seemed to cause the file to be created with some random mode which ended up creating a 'T' file. DHT then proceeded to treat the file as a linkto file instead of a data file, causing the test to fail. Fix: Provide a mode argument to the open() calls.
http://review.gluster.org/#/c/10627/
REVIEW: http://review.gluster.org/10627 (tests: Spurious failure in fop-sanity.t) posted (#2) for review on release-3.7 by N Balachandran (nbalacha)
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.7.0, please open a new bug report. glusterfs-3.7.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution. [1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939 [2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user