Bug 1591621

Summary: Arequal checksum mismatch on older mount
Product: [Community] GlusterFS Reporter: Raghavendra G <rgowdapp>
Component: quick-readAssignee: Raghavendra G <rgowdapp>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: urgent Docs Contact:
Priority: unspecified    
Version: mainlineCC: bugs, pgurusid, rallan, rcyriac, rgowdapp, rhinduja, rhs-bugs, sheggodu, vdas
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: glusterfs-5.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1561393 Environment:
Last Closed: 2018-10-23 15:11:37 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1561393    

Comment 1 Worker Ant 2018-06-15 11:39:25 UTC
REVIEW: https://review.gluster.org/20286 (performance/quick-read: provide an invalidation based on ctime) posted (#1) for review on master by Raghavendra G

Comment 2 Worker Ant 2018-06-18 09:37:41 UTC
COMMIT: https://review.gluster.org/20286 committed in master by "Raghavendra G" <rgowdapp> with a commit message- performance/quick-read: provide an invalidation based on ctime

Quick-read by default uses mtime to identify changes to file
data. However there are applications like rsync which explicitly set
mtime making it unreliable for the purpose of identifying change in
file content. Since ctime also changes when content of a file changes
and it cannot be set explicitly, it becomes suitable for identifying
staleness of cached data. This option makes quick-read to prefer ctime
over mtime to validate its cache. However, using ctime can result in
false positives as ctime changes with just attribute changes like
permission without changes to file data. So, use this option only when
mtime is not reliable.

credits to Kotresh Hiremath Ravishankar <khiremat> for
suggestion on using ctime instead of mtime.

Change-Id: Ib3ae39a3252b2876c8ffe81f471d02a87190e9b9
Signed-off-by: Raghavendra G <rgowdapp>
Updates: bz#1591621

Comment 3 Shyamsundar 2018-10-23 15:11:37 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-5.0, please open a new bug report.

glusterfs-5.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] https://lists.gluster.org/pipermail/announce/2018-October/000115.html
[2] https://www.gluster.org/pipermail/gluster-users/