Bug 1733885 - ctime: Upgrade/Enabling ctime feature wrongly updates older files with latest {a|m|c}time
Summary: ctime: Upgrade/Enabling ctime feature wrongly updates older files with latest...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: ctime
Version: 6
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
Assignee: Kotresh HR
QA Contact:
URL:
Whiteboard:
Depends On: 1593542 1739430
Blocks: 1715422
TreeView+ depends on / blocked
 
Reported: 2019-07-29 06:22 UTC by Kotresh HR
Modified: 2019-08-09 10:07 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1593542
Environment:
Last Closed: 2019-08-06 11:51:31 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Gluster.org Gerrit 23119 0 None Open ctime: Set mdata xattr on legacy files 2019-08-06 11:51:07 UTC
Gluster.org Gerrit 23120 0 None Merged features/utime: Fix mem_put crash 2019-08-06 11:51:30 UTC

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


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