Bug 1130969 - NFS interoperability problem: stripe-xlator removes EOF at end of READDIR
Summary: NFS interoperability problem: stripe-xlator removes EOF at end of READDIR
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: nfs
Version: mainline
Hardware: Unspecified
OS: Unspecified
high
low
Target Milestone: ---
Assignee: Niels de Vos
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1129939 1132390 1132391 1132392
TreeView+ depends on / blocked
 
Reported: 2014-08-18 10:05 UTC by Niels de Vos
Modified: 2015-05-14 17:43 UTC (History)
2 users (show)

Fixed In Version: glusterfs-3.7.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-05-14 17:27:11 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Niels de Vos 2014-08-18 10:05:38 UTC
Description of problem (from Emmanuel Dreyfus):

NFS READDIR replies ar e made of a header, a sequence of
entriers, and a EOF flag. When GlusterFS's NFS server is
used along with stripe xlator, it fails to set the EOF
flag, which violates NFS RFC and confuses some clients.

The bug is caused because nfs xlator sets EOF if it gets
op_errno set to ENOENT. That value is produced in storage
xlator and propagated through server, client, and other
xlators until stripe xlator handles it. stripe only passed
op_errno if op_ret < 0, which is not the case here.

Comment 1 Niels de Vos 2014-08-18 12:01:37 UTC
Example of missing EOF after the "Value Follows: No" marker.

    Frame 834: 1356 bytes on wire (10848 bits), 1356 bytes captured (10848 bits)
    Internet Protocol Version 4, Src: 127.0.0.1 (127.0.0.1), Dst: 127.0.0.1 (127.0.0.1)
    Remote Procedure Call, Type:Reply XID:0xe8e42eee
    Network File System, READDIRPLUS Reply
        [Program Version: 3]
        [V3 Procedure: READDIRPLUS (17)]
        Status: NFS3_OK (0)
        dir_attributes  Directory mode: 0755 uid: 0 gid: 0
            ...
        Verifier: Opaque Data
        Value Follows: Yes
        Entry: name e246f09c-2f58-417b-8c3e-6f7f5307154f
        ...
        Value Follows: Yes
        Entry: name 17cec36b-c553-47bc-a7da-4663f3efbac2
            ...
        Value Follows: No
        EOF: 0

Comment 2 Anand Avati 2014-08-18 12:44:55 UTC
REVIEW: http://review.gluster.org/8493 (NFS: stripe-xlator should pass EOF at end of READDIR) posted (#3) for review on master by Niels de Vos (ndevos)

Comment 3 Niels de Vos 2014-08-18 12:47:31 UTC
Confirmation that EOF is set with the patch:

    Frame 654: 1356 bytes on wire (10848 bits), 1356 bytes captured (10848 bits)
    Internet Protocol Version 4, Src: 127.0.0.1 (127.0.0.1), Dst: 127.0.0.1 (127.0.0.1)
    Transmission Control Protocol, Src Port: nfs (2049), Dst Port: accessnetwork (699), Seq: 19741, Ack: 1777, Len: 1288
    Remote Procedure Call, Type:Reply XID:0xa604116c
    Network File System, READDIRPLUS Reply
        [Program Version: 3]
        [V3 Procedure: READDIRPLUS (17)]
        Status: NFS3_OK (0)
        dir_attributes  Directory mode: 0755 uid: 0 gid: 0
            ...
        Entry: name e246f09c-2f58-417b-8c3e-6f7f5307154f
        ...
        Value Follows: Yes
        Entry: name 17cec36b-c553-47bc-a7da-4663f3efbac2
            ...
        Value Follows: No
        EOF: 1

Comment 4 Anand Avati 2014-08-19 12:23:25 UTC
COMMIT: http://review.gluster.org/8493 committed in master by Niels de Vos (ndevos) 
------
commit 9b5231e5c98b8cfa116838287c7a14042702795f
Author: Niels de Vos <ndevos>
Date:   Mon Aug 18 12:11:35 2014 +0200

    NFS: stripe-xlator should pass EOF at end of READDIR
    
    NFS READDIR replies are made of a header, a sequence of
    entries, and a EOF flag. When GlusterFS's NFS server is
    used along with stripe xlator, it fails to set the EOF
    flag, which violates NFS RFC and confuses some clients.
    
    The bug is caused because nfs xlator sets EOF if it gets
    op_errno set to ENOENT. That value is produced in storage
    xlator and propagated through server, client, and other
    xlators until stripe xlator handles it. stripe only passed
    op_errno if op_ret < 0, which is not the case here. This
    change set adds a special case for that situation to fix
    the problem.
    
    Change-Id: Ie6db94b0515292387cfb04c1e4a9363f34fcd19a
    BUG: 1130969
    Reported-by: Emmanuel Dreyfus <manu>
    Signed-off-by: Niels de Vos <ndevos>
    Reviewed-on: http://review.gluster.org/8493
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Santosh Pradhan <spradhan>
    Reviewed-by: Kaleb KEITHLEY <kkeithle>
    Reviewed-by: Emmanuel Dreyfus <manu>
    Tested-by: Emmanuel Dreyfus <manu>

Comment 5 Niels de Vos 2015-05-14 17:27:11 UTC
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

Comment 6 Niels de Vos 2015-05-14 17:35:32 UTC
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

Comment 7 Niels de Vos 2015-05-14 17:37:54 UTC
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

Comment 8 Niels de Vos 2015-05-14 17:43:18 UTC
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


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