Bug 1507880 - reset-brick commit force failed with glusterd_volume_brickinfo_get Returning -1
Summary: reset-brick commit force failed with glusterd_volume_brickinfo_get Returning -1
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: glusterd
Version: 3.10
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
Assignee: Atin Mukherjee
QA Contact:
URL:
Whiteboard:
Depends On: 1507466
Blocks: 1507172 1507877
TreeView+ depends on / blocked
 
Reported: 2017-10-31 11:22 UTC by Atin Mukherjee
Modified: 2017-12-08 16:46 UTC (History)
7 users (show)

Fixed In Version: glusterfs-3.10.8
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1507466
Environment:
Last Closed: 2017-12-08 16:46:32 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Comment 1 Worker Ant 2017-10-31 11:24:11 UTC
REVIEW: https://review.gluster.org/18600 (glusterd: delete source brick only once in reset-brick commit force) posted (#1) for review on release-3.10 by Atin Mukherjee

Comment 2 Atin Mukherjee 2017-10-31 11:24:52 UTC
Problem Description:
On a two node cluster, reset-brick commit force fails.


Version:
Mainline

Reproducer steps:

1. Create a two node cluster.
2. Create a replica 2 volume, each brick hosted in different nodes
gluster v create test-vol replica 2 172.17.0.2:/tmp/b1 172.17.0.3:/tmp/b1 force
3. gluster v reset-brick test-vol 172.17.0.2:/tmp/b1 start
4. gluster v reset-brick test-vol 172.17.0.2:/tmp/b1  172.17.0.2:/tmp/b1 commit force

Step 4 fails.

--- Additional comment from Atin Mukherjee on 2017-10-30 07:18:07 EDT ---

upstream patch :

--- Additional comment from Atin Mukherjee on 2017-10-30 07:20:20 EDT ---

patch : https://review.gluster.org/#/c/18581

--- Additional comment from Worker Ant on 2017-10-31 07:14:53 EDT ---

COMMIT: https://review.gluster.org/18581 committed in master by  

------------- glusterd: delete source brick only once in reset-brick commit force

While stopping the brick which is to be reset and replaced delete_brick
flag was passed as true which resulted glusterd to free up to source
brick before the actual operation. This results commit force to fail
failing to find the source brickinfo.

Change-Id: I1aa7508eff7cc9c9b5d6f5163f3bb92736d6df44
BUG: 1507466
Signed-off-by: Atin Mukherjee <amukherj>

Comment 3 Worker Ant 2017-10-31 18:08:38 UTC
COMMIT: https://review.gluster.org/18600 committed in release-3.10 by  

------------- glusterd: delete source brick only once in reset-brick commit force

While stopping the brick which is to be reset and replaced delete_brick
flag was passed as true which resulted glusterd to free up to source
brick before the actual operation. This results commit force to fail
failing to find the source brickinfo.

> mainline patch : https://review.gluster.org/#/c/18581/

Change-Id: I1aa7508eff7cc9c9b5d6f5163f3bb92736d6df44
BUG: 1507880
Signed-off-by: Atin Mukherjee <amukherj>
(cherry picked from commit 0fb8acaa6ff80c43e46deac0ce66b29ae0df0ca4)

Comment 4 Shyamsundar 2017-12-08 16:46:32 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.10.8, please open a new bug report.

glusterfs-3.10.8 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://lists.gluster.org/pipermail/announce/2017-December/000086.html
[2] https://www.gluster.org/pipermail/gluster-users/


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