Bug 1274600 - [sharding+geo-rep]: On existing slave mount, reading files fails to show sharded file content
Summary: [sharding+geo-rep]: On existing slave mount, reading files fails to show shar...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: sharding
Version: 3.7.6
Hardware: x86_64
OS: Linux
unspecified
high
Target Milestone: ---
Assignee: Krutika Dhananjay
QA Contact: bugs@gluster.org
URL:
Whiteboard:
Depends On: 1272986
Blocks: Gluster-HC-1 glusterfs-3.7.6
TreeView+ depends on / blocked
 
Reported: 2015-10-23 06:15 UTC by Krutika Dhananjay
Modified: 2015-11-17 06:01 UTC (History)
4 users (show)

Fixed In Version: glusterfs-3.7.6
Doc Type: Bug Fix
Doc Text:
Clone Of: 1272986
Environment:
Last Closed: 2015-11-17 06:01:01 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Krutika Dhananjay 2015-10-23 06:15:27 UTC
+++ This bug was initially created as a clone of Bug #1272986 +++

Description of problem:
=======================

In geo-rep setup, when a data on file increases beyond 4M size the files gets synced to slave but reading file from slave mount only shows the original file content and not the content of sharded chunk. 

Unmounting and remounting the slave volume shows the proper content including sharded file contents. 

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

glusterfs-3.8dev-0.929.gitbece485.el7rhgs.x86_64


How reproducible:
=================

Always


Steps to Reproduce:
===================
1. Create Master and Slave Cluster
2. Create Master and Slave Volume with meta_volume to true and features.shard on. Also enable performance.strict-write-ordering on the master volume
3. Create and Start geo-rep session
4. Mount master and slave volume
5. Create a file with total size less than 4M {I created file_1.9M with 1.9M of size}
6. File should sync to slave. Calculate arequal between master and slave, it should match. Do a diff of file between master and slave, should not show any output.
7. Append the existing file with data to grow total size more than 4M {I appended data of 3.4M to file_1.9M, total size thus becomes 5.2M}
8. Confirm 1 sharded file is created
9. Size at master and slave for file_1.9M is correctly shown as 5.2M but the arequal missmatches. diff shows data is not synced from master to slave. Their is a miss match in sharded files.
10. Umount the slave volume
11. Mount the slave volume again. The arequal matches

Actual results:
===============

On existing slave mount, the file content may be read from some cache


Expected results:
=================

On existing volume, the file should be read as whole including the shareded file content.

--- Additional comment from Vijay Bellur on 2015-10-20 05:31:36 EDT ---

REVIEW: http://review.gluster.org/12400 (features/shard: Refresh cached size when lookup/readdirp detect that xattr value has changed) posted (#1) for review on master by Krutika Dhananjay (kdhananj)

--- Additional comment from Vijay Bellur on 2015-10-20 07:46:40 EDT ---

REVIEW: http://review.gluster.org/12400 (features/shard: Force cache-refresh when lookup/readdirp/stat detect that xattr value has changed) posted (#2) for review on master by Krutika Dhananjay (kdhananj)

Comment 1 Vijay Bellur 2015-10-23 06:21:27 UTC
REVIEW: http://review.gluster.org/12416 (features/shard: Force cache-refresh when lookup/readdirp/stat detect that xattr value has changed) posted (#1) for review on release-3.7 by Krutika Dhananjay (kdhananj)

Comment 2 Vijay Bellur 2015-10-29 08:42:42 UTC
REVIEW: http://review.gluster.org/12457 (features/shard: Force cache-refresh when lookup/readdirp/stat detect that xattr value has changed) posted (#1) for review on release-3.7 by Krutika Dhananjay (kdhananj)

Comment 3 Vijay Bellur 2015-10-30 06:37:00 UTC
COMMIT: http://review.gluster.org/12457 committed in release-3.7 by Pranith Kumar Karampuri (pkarampu) 
------
commit af5e4e4ef75c845149aee4a8fefd50115157a6da
Author: Krutika Dhananjay <kdhananj>
Date:   Tue Oct 20 11:46:10 2015 +0530

    features/shard: Force cache-refresh when lookup/readdirp/stat detect that xattr value has changed
    
            Backport of: http://review.gluster.org/12400
    
    Change-Id: Ifa51979bc530e31d36781759ca62bcac1de7af24
    BUG: 1274600
    Signed-off-by: Krutika Dhananjay <kdhananj>
    Reviewed-on: http://review.gluster.org/12457
    Tested-by: Gluster Build System <jenkins.com>
    Tested-by: NetBSD Build System <jenkins.org>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>

Comment 4 Raghavendra Talur 2015-11-17 06:01:01 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.6, please open a new bug report.

glusterfs-3.7.6 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://www.gluster.org/pipermail/gluster-users/2015-November/024359.html
[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.