[Bugs] [Bug 1396109] Remove-brick status output is showing status of fix-layout instead of original remove-brick status output

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1396109



--- Comment #1 from Nithya Balachandran <nbalacha at redhat.com> ---
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

--- Additional comment from Worker Ant on 2016-10-28 06:10:50 EDT ---

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

--- Additional comment from Worker Ant on 2016-11-16 23:38:10 EST ---

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

--- Additional comment from Worker Ant on 2016-11-17 05:14:19 EST ---

COMMIT: http://review.gluster.org/15749 committed in master by Atin Mukherjee
(amukherj at redhat.com) 
------
commit 35b085ba345cafb2b0ee978a4c4475ab0dcba5a6
Author: N Balachandran <nbalacha at redhat.com>
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 at redhat.com>
    Reviewed-on: http://review.gluster.org/15749
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: Atin Mukherjee <amukherj at redhat.com>

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.