Bug 149851 (IT_67012)

Summary: clusvcmgrd (svclib_filesystem) kills too many processes
Product: [Retired] Red Hat Cluster Suite Reporter: Bastien Nocera <bnocera>
Component: clumanagerAssignee: Lon Hohberger <lhh>
Status: CLOSED ERRATA QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 3CC: cluster-maint, hagedorn, kanderso, rkenna, tao, uthomas
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2005-05-03 20:31:25 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Bastien Nocera 2005-02-28 10:43:55 UTC
clusvcmgrd kills too many processes when code wrongly matches too many
entries (/usr/lib/clumanager/services/svclib_filesystem:182). For
example, processes accessing:
/var/spool/mail
/var/spool/mail_bak

Both will be killed when matching /var/spool/mail:
          done < <(lsof -b 2>/dev/null | \
                    grep "$mp" | \
                    awk '{print $1,$2,$3}' | \
                    sort -u -k 1,3)

Patch from installed (by uthomas):

--- ./usr/lib/clumanager/services/svclib_filesystem.old 2005-02-25
17:09:43.000000000 +0000
+++ ./usr/lib/clumanager/services/svclib_filesystem     2005-02-28
09:23:07.508283144 +0000
@@ -181,7 +181,7 @@
               kill -TERM $pid
             fi
           done < <(lsof -b 2>/dev/null | \
-                    grep "$mp" | \
+                    grep -E "$mp(/.*|)\$" | \
                    awk '{print $1,$2,$3}' | \
                    sort -u -k 1,3)
         elif [ -n "$have_fuser" ]; then

Comment 2 Lon Hohberger 2005-02-28 15:50:26 UTC
Patch looks clean.

Comment 3 Lon Hohberger 2005-02-28 15:51:07 UTC
Accepting bug.

Comment 5 Lon Hohberger 2005-02-28 16:07:03 UTC
Setting to RHCS.

Comment 7 Lon Hohberger 2005-03-02 06:32:40 UTC
in 1.2.26 for testing

Comment 10 Lon Hohberger 2005-05-03 20:31:25 UTC
Works for me on the beta bits.

Comment 11 Jay Turner 2005-05-25 16:41:03 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on the solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2005-047.html