Bug 1389697

Summary:	Remove-brick status output is showing status of fix-layout instead of original remove-brick status output
Product:	[Community] GlusterFS	Reporter:	Nithya Balachandran <nbalacha>
Component:	distribute	Assignee:	Nithya Balachandran <nbalacha>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	bugs, rhs-bugs, storage-qa-internal, tdesala
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.10.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1386127
Clones:	1396109 (view as bug list)		Environment:
Last Closed:	2017-03-06 17:31:48 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:
Bug Blocks:	1386127, 1396109

Comment 1 Nithya Balachandran 2016-10-28 09:24:10 UTC

Steps to reproduce the issue:

1. On a 2 node cluster, create a volume with 1 brick on each node.
2. From node1, run 
gluster v rebalance <volname> fix-layout start

3. Once the fix-layout has completed,  from node1, run 
gluster v remove-brick <volname> <brick on node2> start

4. On node1, run
gluster v remove-brick <volname> <brick on node2> status


This will print the fix-layout output.

Running the command on node2 prints the output correctly.
gluster v remove-brick <volname> <brick on node2> status

Comment 2 Worker Ant 2016-10-28 10:10:50 UTC

REVIEW: http://review.gluster.org/15749 (cli/rebalance: remove brick status is incorrect) posted (#1) for review on master by N Balachandran (nbalacha)

Comment 3 Worker Ant 2016-11-17 04:38:10 UTC

REVIEW: http://review.gluster.org/15749 (cli/rebalance: remove brick status is incorrect) posted (#2) for review on master by N Balachandran (nbalacha)

Comment 4 Worker Ant 2016-11-17 10:14:19 UTC

COMMIT: http://review.gluster.org/15749 committed in master by Atin Mukherjee (amukherj) 
------
commit 35b085ba345cafb2b0ee978a4c4475ab0dcba5a6
Author: N Balachandran <nbalacha>
Date:   Fri Oct 28 15:21:52 2016 +0530

    cli/rebalance: remove brick status is incorrect
    
    If a remove brick operation is preceded by a fix-layout,
    running remove-brick status on a node which does not
    contain any of the bricks that were removed displays
    fix-layout status.
    
    The defrag_cmd variable was not updated in glusterd
    for the nodes not hosting removed bricks causing the
    status parsing to go wrong. This is now updated.
    Also made minor modifications to the spacing in
    the fix-layout status output.
    
    Change-Id: Ib735ce26be7434cd71b76e4c33d9b0648d0530db
    BUG: 1389697
    Signed-off-by: N Balachandran <nbalacha>
    Reviewed-on: http://review.gluster.org/15749
    Smoke: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Atin Mukherjee <amukherj>

Comment 5 Shyamsundar 2017-03-06 17:31:48 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.0, please open a new bug report.

glusterfs-3.10.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://lists.gluster.org/pipermail/gluster-users/2017-February/030119.html
[2] https://www.gluster.org/pipermail/gluster-users/