Bug 1260051

Summary: DHT: Few files are missing after remove-brick operation
Product: [Community] GlusterFS Reporter: Susant Kumar Palai <spalai>
Component: distributeAssignee: bugs <bugs>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: mainlineCC: bugs, byarlaga, rmekala, spalai, storage-qa-internal
Target Milestone: ---Keywords: ZStream
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: glusterfs-3.8rc2 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1259750
: 1262197 (view as bug list) Environment:
Last Closed: 2016-06-16 13:35:09 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: 1259750    
Bug Blocks: 1262197    

Comment 1 Vijay Bellur 2015-09-04 10:12:56 UTC
REVIEW: http://review.gluster.org/12105 (dht/remove-brick: Avoid data loss for hard link migration) posted (#1) for review on master by Susant Palai (spalai)

Comment 2 Vijay Bellur 2015-09-09 10:37:33 UTC
REVIEW: http://review.gluster.org/12105 (dht/remove-brick: Avoid data loss for hard link migration) posted (#2) for review on master by Susant Palai (spalai)

Comment 3 Vijay Bellur 2015-09-09 12:04:14 UTC
REVIEW: http://review.gluster.org/12105 (dht/remove-brick: Avoid data loss for hard link migration) posted (#3) for review on master by Susant Palai (spalai)

Comment 4 Vijay Bellur 2015-09-11 10:16:43 UTC
REVIEW: http://review.gluster.org/12160 (dht/cluster: Avoid crash if local is NULL) posted (#1) for review on master by Susant Palai (spalai)

Comment 5 Vijay Bellur 2015-09-11 12:56:35 UTC
REVIEW: http://review.gluster.org/12160 (dht/cluster: Avoid crash if local is NULL) posted (#2) for review on master by Susant Palai (spalai)

Comment 6 Vijay Bellur 2015-09-14 05:48:54 UTC
COMMIT: http://review.gluster.org/12160 committed in master by Raghavendra G (rgowdapp) 
------
commit c31ac73cd8e9ca4b872a48667e09019991dea00b
Author: Susant Palai <spalai>
Date:   Fri Sep 11 08:52:44 2015 -0400

    dht/cluster: Avoid crash if local is NULL
    
    This patch addresses crash handling if local is NULL. In addition to that,
    we were not unwinding if no lock is taken in dht_linkfile_create_cbk(create/mknod).
    This patch handles that also.
    
    Change-Id: Ibcff317f10d60e7865fd7ffb9479b3af53c9ef17
    BUG: 1260051
    Signed-off-by: Susant Palai <spalai>
    Reviewed-on: http://review.gluster.org/12160
    Reviewed-by: N Balachandran <nbalacha>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Raghavendra G <rgowdapp>

Comment 7 Kaleb KEITHLEY 2015-10-22 17:00:12 UTC
changing version to mainline in order to retire pre-release.

If you know the appropriate, correct version for this bug, please
set it.

Comment 8 Kaleb KEITHLEY 2015-10-22 17:06:51 UTC
changing version to mainline in order to retire pre-release.

If you know the appropriate, correct version for this bug, please
set it.

Comment 9 Kaleb KEITHLEY 2015-10-22 17:11:18 UTC
changing version to mainline in order to retire pre-release.

If you know the appropriate, correct version for this bug, please
set it.

Comment 10 Niels de Vos 2016-06-16 13:35:09 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.8.0, please open a new bug report.

glusterfs-3.8.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] http://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user