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

Bug 1243647

Summary:	Disperse volume : data corruption with appending writes in 8+4 config
Product:	[Community] GlusterFS	Reporter:	Pranith Kumar K <pkarampu>
Component:	disperse	Assignee:	bugs <bugs>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	3.7.3	CC:	asrivast, bugs, byarlaga, gluster-bugs, nsathyan, pkarampu
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.7.3	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1232678	Environment:
Last Closed:		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:	1230513, 1232678
Bug Blocks:	1223636

Comment 1 Anand Avati 2015-07-16 03:26:42 UTC

REVIEW: http://review.gluster.org/11682 (cluster/ec: Don't read from bad subvols) posted (#1) for review on release-3.7 by Pranith Kumar Karampuri (pkarampu)

Comment 2 Anand Avati 2015-07-16 03:27:13 UTC

REVIEW: http://review.gluster.org/11685 (cluster/ec: Don't read from bricks that are healing) posted (#1) for review on release-3.7 by Pranith Kumar Karampuri (pkarampu)

Comment 3 Anand Avati 2015-07-16 03:27:15 UTC

REVIEW: http://review.gluster.org/11686 (cluster/ec: Prevent data corruptions) posted (#1) for review on release-3.7 by Pranith Kumar Karampuri (pkarampu)

Comment 4 Anand Avati 2015-07-21 11:55:47 UTC

REVIEW: http://review.gluster.org/11682 (cluster/ec: Don't read from bad subvols) posted (#2) for review on release-3.7 by Xavier Hernandez (xhernandez)

Comment 5 Anand Avati 2015-07-21 11:56:46 UTC

REVIEW: http://review.gluster.org/11685 (cluster/ec: Don't read from bricks that are healing) posted (#2) for review on release-3.7 by Xavier Hernandez (xhernandez)

Comment 6 Anand Avati 2015-07-21 11:57:11 UTC

REVIEW: http://review.gluster.org/11686 (cluster/ec: Prevent data corruptions) posted (#2) for review on release-3.7 by Xavier Hernandez (xhernandez)

Comment 7 Anand Avati 2015-07-21 21:42:50 UTC

COMMIT: http://review.gluster.org/11682 committed in release-3.7 by Xavier Hernandez (xhernandez) 
------
commit 1a7d49cfecd75b97641ab9261ee3c930bd51e895
Author: Pranith Kumar K <pkarampu>
Date:   Mon Jul 6 14:23:56 2015 +0530

    cluster/ec: Don't read from bad subvols
    
     >Change-Id: Ic22813371faca4e8198c9b0b20518e68d275f3c1
     >BUG: 1232678
     >Signed-off-by: Pranith Kumar K <pkarampu>
     >Reviewed-on: http://review.gluster.org/11531
     >Reviewed-by: Xavier Hernandez <xhernandez>
     >Tested-by: NetBSD Build System <jenkins.org>
     >Tested-by: Gluster Build System <jenkins.com>
    
    BUG: 1243647
    Change-Id: Ic8570710a16715322bc0be59367007567f6438cd
    Signed-off-by: Pranith Kumar K <pkarampu>
    Reviewed-on: http://review.gluster.org/11682
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Xavier Hernandez <xhernandez>

Comment 8 Anand Avati 2015-07-22 06:26:53 UTC

COMMIT: http://review.gluster.org/11685 committed in release-3.7 by Pranith Kumar Karampuri (pkarampu) 
------
commit 42b834669a08547de953eb42d6cc1601461c7fbb
Author: Pranith Kumar K <pkarampu>
Date:   Wed Jul 8 17:52:11 2015 +0530

    cluster/ec: Don't read from bricks that are healing
    
     >BUG: 1232678
     >Change-Id: I35503039e4723cf7f33d6797f0ba90dd0aca130b
     >Signed-off-by: Pranith Kumar K <pkarampu>
     >Reviewed-on: http://review.gluster.org/11580
     >Tested-by: NetBSD Build System <jenkins.org>
     >Tested-by: Gluster Build System <jenkins.com>
     >Reviewed-by: Xavier Hernandez <xhernandez>
    
    BUG: 1243647
    Change-Id: I4eb45197a5a8d9652eded37ba1e67d9ea745a583
    Signed-off-by: Pranith Kumar K <pkarampu>
    Reviewed-on: http://review.gluster.org/11685
    Reviewed-by: Xavier Hernandez <xhernandez>
    Tested-by: Gluster Build System <jenkins.com>

Comment 9 Anand Avati 2015-07-22 06:27:49 UTC

COMMIT: http://review.gluster.org/11686 committed in release-3.7 by Pranith Kumar Karampuri (pkarampu) 
------
commit b8a079b17f5aeb659ca35b9c7ec4b0ac7dc80406
Author: Pranith Kumar K <pkarampu>
Date:   Mon Jul 13 00:53:20 2015 +0530

    cluster/ec: Prevent data corruptions
    
    - On lock reuse preserve 'healing' bits
    - Don't set ctx->size outside locks in healing code
    - Allow xattrop internal fops also on the fop->mask.
    
     >Change-Id: I6b76da5d7ebe367d8f3552cbf9fd18e556f2a171
     >BUG: 1232678
     >Signed-off-by: Pranith Kumar K <pkarampu>
     >Reviewed-on: http://review.gluster.org/11640
     >Tested-by: NetBSD Build System <jenkins.org>
     >Tested-by: Gluster Build System <jenkins.com>
     >Reviewed-by: Xavier Hernandez <xhernandez>
    
    BUG: 1243647
    Change-Id: I1b3828e4d4a863b84b2c4e732e7965d1302cea47
    Signed-off-by: Pranith Kumar K <pkarampu>
    Reviewed-on: http://review.gluster.org/11686
    Reviewed-by: Xavier Hernandez <xhernandez>
    Tested-by: Gluster Build System <jenkins.com>

Comment 10 Kaushal 2015-07-30 09:47:41 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.3, please open a new bug report.

glusterfs-3.7.3 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://thread.gmane.org/gmane.comp.file-systems.gluster.devel/12078
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 11 Kaushal 2015-07-30 09:47:42 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.3, please open a new bug report.

glusterfs-3.7.3 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://thread.gmane.org/gmane.comp.file-systems.gluster.devel/12078
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 12 Kaushal 2015-07-30 09:47:43 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.3, please open a new bug report.

glusterfs-3.7.3 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://thread.gmane.org/gmane.comp.file-systems.gluster.devel/12078
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user