Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1733885

Summary: ctime: Upgrade/Enabling ctime feature wrongly updates older files with latest {a|m|c}time
Product: [Community] GlusterFS Reporter: Kotresh HR <khiremat>
Component: ctimeAssignee: Kotresh HR <khiremat>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: high Docs Contact:
Priority: high    
Version: 6CC: atumball, bugs, pasik, pkarampu
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1593542 Environment:
Last Closed: 2019-08-06 11:51:31 UTC Type: ---
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: 1593542, 1739430    
Bug Blocks: 1715422    

Description Kotresh HR 2019-07-29 06:22:59 UTC 
+++ This bug was initially created as a clone of Bug #1593542 +++

Description of problem:
Upgrade scenario: Currently for older files, the ctime gets updated during {a|m|c}time modification fop and eventually becomes consistent. 
With any {a|m|c}time modification, the ctime is initialized with latest time which is incorrect. So how do we handle this upgrade scenario.

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

How reproducible:
Always

Steps to Reproduce:
1.  Create EC/replica volume, mount it, create a file.
2.  Enable ctime feature
3.  touch the created file {m|a|c}time will be latest. Only access time should have been updated.

Actual results:
 {a|m|c}time gets updated.

Expected results:
 Only access time should have been updated.


Additional info:

--- Additional comment from Worker Ant on 2019-06-24 19:11:36 UTC ---

REVIEW: https://review.gluster.org/22936 (ctime: Set mdata xattr on legacy files) posted (#1) for review on master by Kotresh HR

--- Additional comment from Worker Ant on 2019-07-22 06:57:08 UTC ---

REVIEW: https://review.gluster.org/22936 (ctime: Set mdata xattr on legacy files) merged (#14) on master by Atin Mukherjee

--- Additional comment from Worker Ant on 2019-07-22 15:30:46 UTC ---

REVIEW: https://review.gluster.org/23091 (features/utime: Fix mem_put crash) posted (#1) for review on master by Pranith Kumar Karampuri

--- Additional comment from Worker Ant on 2019-07-23 01:29:55 UTC ---

REVIEW: https://review.gluster.org/23091 (features/utime: Fix mem_put crash) merged (#1) on master by Pranith Kumar Karampuri
Comment 1 Worker Ant 2019-07-29 07:11:44 UTC 
REVIEW: https://review.gluster.org/23119 (ctime: Set mdata xattr on legacy files) posted (#1) for review on release-6 by Kotresh HR
Comment 2 Worker Ant 2019-07-29 07:12:53 UTC 
REVIEW: https://review.gluster.org/23120 (features/utime: Fix mem_put crash) posted (#1) for review on release-6 by Kotresh HR
Comment 3 Worker Ant 2019-08-06 11:51:08 UTC 
REVIEW: https://review.gluster.org/23119 (ctime: Set mdata xattr on legacy files) merged (#2) on release-6 by Kotresh HR
Comment 4 Worker Ant 2019-08-06 11:51:31 UTC 
REVIEW: https://review.gluster.org/23120 (features/utime: Fix mem_put crash) merged (#2) on release-6 by Kotresh HR