Bug 1347657 - EOF flag not set in NFSv3 read replies for striped volumes
Summary: EOF flag not set in NFSv3 read replies for striped volumes
Keywords:
Status: CLOSED DUPLICATE of bug 1208384
Alias: None
Product: GlusterFS
Classification: Community
Component: stripe
Version: 3.6.8
Hardware: Unspecified
OS: FreeBSD
unspecified
unspecified
Target Milestone: ---
Assignee: Niels de Vos
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-06-17 10:54 UTC by 2510
Modified: 2016-06-21 12:28 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-06-21 12:26:12 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)
packet dump file (1.70 KB, application/octet-stream)
2016-06-17 10:54 UTC, 2510
no flags Details
Experimental patch (1.72 KB, patch)
2016-06-17 10:55 UTC, 2510
no flags Details | Diff

Description 2510 2016-06-17 10:54:48 UTC
Created attachment 1169028 [details]
packet dump file

Description of problem:
With posix-stripe, glusterfs does not set EOF flag for NFSv3 reads.

Version-Release number of selected component (if applicable):

How reproducible:
Always.

Steps to Reproduce:
1. Create striped volume.
2. On VMware ESXi, create NFS datastore connected to volume created above.
3. Create a new virtual disk on it.

Actual results:
EOF is not set in NFSv3 read reply.
(and ESXi fails to create a virtual disk.)

Expected results:
EOF is set.

Additional info:
Attaches a packet dump(pcap) and a patch.
ec(disperse) and dht(replicated) translator correctly set EOF.

Comment 1 2510 2016-06-17 10:55:39 UTC
Created attachment 1169030 [details]
Experimental patch

Comment 2 Niels de Vos 2016-06-21 12:24:20 UTC
This should have been fixed in current 3.7 versions and newer. The following patch addressed it:

  http://review.gluster.org/10142

Comment 3 Niels de Vos 2016-06-21 12:26:12 UTC
It seems that there already is a bug for 3.6.x. We'll use that one to get the change included.

*** This bug has been marked as a duplicate of bug 1208384 ***

Comment 4 2510 2016-06-21 12:28:58 UTC
Sorry for duplicate.
Thanks.


Note You need to log in before you can comment on or make changes to this bug.