Bug 1318289 - [RFE] Add arbiter brick hotplug
Summary: [RFE] Add arbiter brick hotplug
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: arbiter
Version: mainline
Hardware: All
OS: All
unspecified
low
Target Milestone: ---
Assignee: Ravishankar N
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1337387
TreeView+ depends on / blocked
 
Reported: 2016-03-16 12:49 UTC by Oleksandr Natalenko
Modified: 2016-08-26 12:18 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
: 1337387 (view as bug list)
Environment:
Last Closed: 2016-08-26 12:18:20 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Oleksandr Natalenko 2016-03-16 12:49:45 UTC
For existing "replica 2" setup to be converted into "replica 3 arbiter 1" it is necessary to create new volume and copy data to it. However, it is quite complex for large production setup with terabytes of data.

In discussion held with GlusterFS developers in IRC channel it was stated that arbiter brick hotplug should be the feature that is easy to implement.

It would be nice to see such a feature to be implemented for 3.7.x.

Comment 1 Vijay Bellur 2016-04-30 08:29:19 UTC
REVIEW: http://review.gluster.org/14126 (cli/glusterd: Extend add-brick for arbiter volumes) posted (#1) for review on master by Ravishankar N (ravishankar)

Comment 2 Vijay Bellur 2016-04-30 10:15:51 UTC
REVIEW: http://review.gluster.org/14126 (cli/glusterd: Extend add-brick for arbiter volumes) posted (#2) for review on master by Ravishankar N (ravishankar)

Comment 3 Vijay Bellur 2016-05-08 17:49:37 UTC
REVIEW: http://review.gluster.org/14126 (cli/glusterd: add/remove brick fixes for arbiter volumes) posted (#3) for review on master by Ravishankar N (ravishankar)

Comment 4 Vijay Bellur 2016-05-11 05:05:20 UTC
REVIEW: http://review.gluster.org/14126 (cli/glusterd: add/remove brick fixes for arbiter volumes) posted (#4) for review on master by Ravishankar N (ravishankar)

Comment 5 Vijay Bellur 2016-05-17 09:40:26 UTC
REVIEW: http://review.gluster.org/14126 (cli/glusterd: add/remove brick fixes for arbiter volumes) posted (#5) for review on master by Ravishankar N (ravishankar)

Comment 6 Vijay Bellur 2016-05-17 09:47:04 UTC
REVIEW: http://review.gluster.org/14126 (cli/glusterd: add/remove brick fixes for arbiter volumes) posted (#6) for review on master by Ravishankar N (ravishankar)

Comment 7 Vijay Bellur 2016-05-19 04:30:33 UTC
REVIEW: http://review.gluster.org/14126 (cli/glusterd: add/remove brick fixes for arbiter volumes) posted (#7) for review on master by Ravishankar N (ravishankar)

Comment 8 Vijay Bellur 2016-05-19 16:40:09 UTC
COMMIT: http://review.gluster.org/14126 committed in master by Atin Mukherjee (amukherj) 
------
commit 61c1b2cee973b11897a37d508910012e616033bc
Author: Ravishankar N <ravishankar>
Date:   Fri Apr 29 17:41:18 2016 +0530

    cli/glusterd: add/remove brick fixes for arbiter volumes
    
    1.Provide a command to convert replica 2 volumes to arbiter volumes.
    Existing self-heal logic will automatically heal the file hierarchy into
    the arbiter brick, the progress of which can be monitored using the
    heal info command.
    
    Syntax: gluster volume add-brick <VOLNAME> replica 3 arbiter 1
    <HOST:arbiter-brick-path>
    
    2. Add checks when removing bricks from arbiter volumes:
    - When converting from arbiter to replica 2 volume, allow only arbiter
      brick to be removed.
    - When converting from arbiter to plain distribute volume, allow only if
      arbiter is one of the bricks that is removed.
    
    3. Some clean-up:
    - Use GD_MSG_DICT_GET_SUCCESS instead of GD_MSG_DICT_GET_FAILED to
    log messages that are not failures.
    - Remove unused variable `brick_list`
    - Move 'brickinfo->group' related functions to glusted-utils.
    
    Change-Id: Ic87b8c7e4d7d3ab03f93e7b9f372b314d80947ce
    BUG: 1318289
    Signed-off-by: Ravishankar N <ravishankar>
    Reviewed-on: http://review.gluster.org/14126
    Smoke: Gluster Build System <jenkins.com>
    CentOS-regression: Gluster Build System <jenkins.com>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Reviewed-by: Atin Mukherjee <amukherj>


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