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

Bug 1149725

Summary:	Self-heal on dispersed volumes does not restore the correct date
Product:	[Community] GlusterFS	Reporter:	Xavi Hernandez <jahernan>
Component:	disperse	Assignee:	Xavi Hernandez <jahernan>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	3.6.0	CC:	bugs, gluster-bugs
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.6.1	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1149723	Environment:
Last Closed:	2014-11-10 15:14:07 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:	1149723
Bug Blocks:	1117822

Description Xavi Hernandez 2014-10-06 14:23:35 UTC

+++ This bug was initially created as a clone of Bug #1149723 +++

Description of problem:

When a file is missing or corrupt on a brick of a dispersed volume, self-heal repairs it, but the modification time is not updated correctly (it contains the time at which self-heal healed the file, not the original modification time of the file).

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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Anand Avati 2014-10-06 14:42:55 UTC

REVIEW: http://review.gluster.org/8906 (ec: Fix incorrect management of healed bricks) posted (#1) for review on release-3.6 by Xavier Hernandez (xhernandez)

Comment 2 Anand Avati 2014-10-08 09:09:39 UTC

REVIEW: http://review.gluster.org/8906 (ec: Fix incorrect management of healed bricks) posted (#2) for review on release-3.6 by Xavier Hernandez (xhernandez)

Comment 3 Anand Avati 2014-10-08 12:36:53 UTC

REVIEW: http://review.gluster.org/8906 (ec: Fix incorrect management of healed bricks) posted (#3) for review on release-3.6 by Xavier Hernandez (xhernandez)

Comment 4 Anand Avati 2014-10-20 10:40:34 UTC

COMMIT: http://review.gluster.org/8906 committed in release-3.6 by Vijay Bellur (vbellur) 
------
commit b2a1be0f7bf5e19b266ee57c3287ee9f9170e2f7
Author: Xavier Hernandez <xhernandez>
Date:   Mon Oct 6 16:34:58 2014 +0200

    ec: Fix incorrect management of healed bricks
    
    The final lookup made to restore final file attributes after a self-heal
    did clear the mask of bad bricks, causing that the final setattr won't
    modify any brick at all. This caused that some attriutes, specially the
    modification time of the file didn't get updated properly.
    
    Now the mask of healed bricks is saved before doing the last lookup.
    It's also used to correctly report the repaired bricks.
    
    This is a backport of http://review.gluster.org/8905/
    
    Change-Id: Ib94083c9e1b562515dfb54f9574120f1f031dccc
    BUG: 1149725
    Signed-off-by: Xavier Hernandez <xhernandez>
    Reviewed-on: http://review.gluster.org/8906
    Reviewed-by: Dan Lambright <dlambrig>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Vijay Bellur <vbellur>

Comment 5 Niels de Vos 2014-11-10 15:14:07 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.6.1, please reopen this bug report.

glusterfs-3.6.1 has been announced [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://supercolony.gluster.org/pipermail/gluster-users/2014-November/019410.html
[2] http://supercolony.gluster.org/mailman/listinfo/gluster-users