Bug 1342796

Summary: self heal deamon killed due to oom kills on a dist-disperse volume using nfs ganesha
Product: [Community] GlusterFS Reporter: Ashish Pandey <aspandey>
Component: disperseAssignee: Ashish Pandey <aspandey>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: urgent Docs Contact:
Priority: unspecified    
Version: mainlineCC: amukherj, bugs, mzywusko, nchilaka, pkarampu, rcyriac, rhinduja
Target Milestone: ---Keywords: ZStream
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 3.10.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1342426
: 1342954 (view as bug list) Environment:
Last Closed: 2017-03-08 08:29:38 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: 1342426    
Bug Blocks: 1342954, 1342964    

Comment 1 Vijay Bellur 2016-06-06 05:52:12 UTC 
REVIEW: http://review.gluster.org/14649 (cluster/ec: Restrict the launch of replace brick heal) posted (#1) for review on master by Ashish Pandey (aspandey)
Comment 2 Vijay Bellur 2016-06-06 21:34:13 UTC 
COMMIT: http://review.gluster.org/14649 committed in master by Jeff Darcy (jdarcy) 
------
commit eb7ba8984c5e66b57d592f9da606fbdd1133df3b
Author: Ashish Pandey <aspandey>
Date:   Mon Jun 6 10:17:54 2016 +0530

    cluster/ec: Restrict the launch of replace brick heal
    
    Problem: When features.cache-invalidation is ON, a lot of
    ec_notify function gets called which leads to launch of
    too many heals. This leads to no heal completion,
    which causes accumulation of heals.
    
    Solution: ec_launch_replace_heal should not be launch
    for every event. Replace brick will trigger a child up
    event and then only this heal function should be called.
    
    Change-Id: I57b44c6a279d57230daea1d93229be6069245b7d
    BUG: 1342796
    Signed-off-by: Ashish Pandey <aspandey>
    Reviewed-on: http://review.gluster.org/14649
    Reviewed-by: Xavier Hernandez <xhernandez>
    Smoke: Gluster Build System <jenkins.com>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.com>