Bug 1318289

Summary:	[RFE] Add arbiter brick hotplug
Product:	[Community] GlusterFS	Reporter:	Oleksandr Natalenko <oleksandr>
Component:	arbiter	Assignee:	Ravishankar N <ravishankar>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	atalur, bugs, kdhananj, pkarampu, ravishankar, sasundar
Target Milestone:	---	Keywords:	FutureFeature, Triaged
Target Release:	---
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:		Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:
Clones:	1337387 (view as bug list)		Environment:
Last Closed:	2016-08-26 12:18:20 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:	1337387

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>