[Bugs] [Bug 1313921] Incorrect file size on mount if stat is served from the arbiter brick.

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1313921



--- Comment #7 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/13609 committed in release-3.7 by Pranith
Kumar Karampuri (pkarampu at redhat.com) 
------
commit 10e091508ca6e5af815fce612be48287d354a01b
Author: Ravishankar N <ravishankar at redhat.com>
Date:   Mon Feb 29 05:16:50 2016 +0000

    afr: do not set arbiter as a readable subvol in inode context

    Problem:
    If afr_lookup_done() or afr_read_subvol_select_by_policy() chooses the
    arbiter brick to serve the stat() data, file size will be reported as
    zero from the mount, despite other data bricks being available. This can
    break programs like tar which use the stat info to decide how much to read.

    Fix:
    In the inode-context, mark arbiter as a non-readable subvol for both
    data and metadata.

    It it to be noted that by making this fix, we are *not* going to serve
    metadata FOPS anymore from the arbiter brick despite the brick storing
    the metadata. It makes sense to do this because the ever increasing
    over-loaded FOPs (getxattr returning stat data etc.) and compound FOPS
    in gluster will otherwise make it difficult to add checks in code to
    handle corner cases.

     >Change-Id: Ic60b25d77fd05e0897481b7fcb3716d4f2101001
     >BUG: 1310171
     >Signed-off-by: Ravishankar N <ravishankar at redhat.com>
     >Reported-by: Mat Clayton <mat at mixcloud.com>
     >Reviewed-on: http://review.gluster.org/13539
     >Reviewed-by: Anuradha Talur <atalur at redhat.com>
     >Reviewed-by: Krutika Dhananjay <kdhananj at redhat.com>
     >Smoke: Gluster Build System <jenkins at build.gluster.com>
     >NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
     >CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
     >Reviewed-by: Jeff Darcy <jdarcy at redhat.com>

    BUG: 1313921
    Change-Id: I07fc08d633ca2af48f7354454bc2ab75cedb850a
    Signed-off-by: Ravishankar N <ravishankar at redhat.com>
    Reviewed-on: http://review.gluster.org/13609
    Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>
    Tested-by: Pranith Kumar Karampuri <pkarampu at redhat.com>
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=nr9yYikyu2&a=cc_unsubscribe