Bug 1352019

Summary: RFE: Move throttling code to libglusterfs from bitrot
Product: [Community] GlusterFS Reporter: Kotresh HR <khiremat>
Component: bitrotAssignee: Kotresh HR <khiremat>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: medium Docs Contact: bugs <bugs>
Priority: medium    
Version: mainlineCC: bugs
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: v3.10.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1357514 (view as bug list) Environment:
Last Closed: 2017-03-08 09:30:49 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: 1357514, 1357535    

Description Kotresh HR 2016-07-01 12:17:45 UTC
Description of problem:
Throttling is feature by itself which any component can consume, hence it more suitable to be part of libglusterfs than to be of bitrot.

Version-Release number of selected component (if applicable):
mainline

How reproducible:
NA

Steps to Reproduce:
NA

Actual results:
Throttling code is part of bitrot

Expected results:
Throttling code should be part of libglusterfs

Additional info:

Comment 1 Vijay Bellur 2016-07-01 12:40:11 UTC
REVIEW: http://review.gluster.org/14846 (features/bitrot: Move throttling code to libgluterfs) posted (#1) for review on master by Kotresh HR (khiremat)

Comment 2 Vijay Bellur 2016-07-11 06:23:32 UTC
REVIEW: http://review.gluster.org/14846 (features/bitrot: Move throttling code to libgluterfs) posted (#2) for review on master by Kotresh HR (khiremat)

Comment 3 Vijay Bellur 2016-07-11 09:16:07 UTC
REVIEW: http://review.gluster.org/14846 (features/bitrot: Move throttling code to libglusterfs) posted (#3) for review on master by Kotresh HR (khiremat)

Comment 4 Vijay Bellur 2016-07-15 09:21:11 UTC
REVIEW: http://review.gluster.org/14846 (features/bitrot: Move throttling code to libglusterfs) posted (#4) for review on master by Kotresh HR (khiremat)

Comment 5 Vijay Bellur 2016-07-18 12:03:22 UTC
COMMIT: http://review.gluster.org/14846 committed in master by Jeff Darcy (jdarcy) 
------
commit 713d7d080df51bf19ce29cf5d682bc006e1c3a19
Author: Kotresh HR <khiremat>
Date:   Fri Jul 1 15:54:07 2016 +0530

    features/bitrot: Move throttling code to libglusterfs
    
    Since throttling is a separate feature by itself,
    move throttling code to libglusterfs.
    
    Change-Id: If9b99885ceb46e5b1865a4af18b2a2caecf59972
    BUG: 1352019
    Signed-off-by: Kotresh HR <khiremat>
    Reviewed-on: http://review.gluster.org/14846
    Smoke: Gluster Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Reviewed-by: Ravishankar N <ravishankar>
    Reviewed-by: Jeff Darcy <jdarcy>