Bug 1081013 - glusterd needs xfsprogs and e2fsprogs packages
Summary: glusterd needs xfsprogs and e2fsprogs packages
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: glusterd
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Kaleb KEITHLEY
QA Contact:
URL:
Whiteboard:
: 1081605 (view as bug list)
Depends On:
Blocks: 1081016 1081018
TreeView+ depends on / blocked
 
Reported: 2014-03-26 13:35 UTC by Kaleb KEITHLEY
Modified: 2014-11-11 08:29 UTC (History)
3 users (show)

Fixed In Version: glusterfs-3.6.0beta1
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1081016 1081018 1081605 (view as bug list)
Environment:
Last Closed: 2014-11-11 08:29:17 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Kaleb KEITHLEY 2014-03-26 13:35:16 UTC
email on gluster-users mailing list:


Subject: Re: [Gluster-users] Gluster 3.4.2 on Redhat 6.5

I’ve got to the bottom of it…. By running glusterd in foreground with debug enabled I was able to see two error messages when the command was being run… it appears that it was requiring the xfsprogs package which I did not have installed. Once I installed it it appears that zombie processes are no longer being “created”.

Comment 1 Anand Avati 2014-03-26 13:52:24 UTC
REVIEW: http://review.gluster.org/7349 (glusterfs.spec.in: glusterd requires e2fsprogs and xfsprogs) posted (#1) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 2 Anand Avati 2014-03-26 15:00:10 UTC
REVIEW: http://review.gluster.org/7349 (glusterfs.spec.in: glusterd requires e2fsprogs and xfsprogs) posted (#2) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 3 Anand Avati 2014-03-27 18:53:08 UTC
REVIEW: http://review.gluster.org/7360 (glusterd: call runner_end even if runner_start fails) posted (#1) for review on master by Jeff Darcy (jdarcy)

Comment 4 Anand Avati 2014-03-27 20:42:12 UTC
REVIEW: http://review.gluster.org/7361 (xlators/mgmt: don't leave zombies if required programs aren't installed) posted (#1) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 5 Kaleb KEITHLEY 2014-04-08 12:39:22 UTC
*** Bug 1081605 has been marked as a duplicate of this bug. ***

Comment 6 Anand Avati 2014-04-08 22:18:34 UTC
REVIEW: http://review.gluster.org/7361 (xlators/mgmt: don't leave zombies if required programs aren't installed) posted (#2) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 7 Anand Avati 2014-04-09 08:20:53 UTC
COMMIT: http://review.gluster.org/7360 committed in master by Anand Avati (avati) 
------
commit aa199093fdf37dcd87a73cea83f9b9164d5800c5
Author: Jeff Darcy <jdarcy>
Date:   Thu Mar 27 18:51:08 2014 +0000

    glusterd: call runner_end even if runner_start fails
    
    Change-Id: I5eca01a131307ba3be2aed4922eea73025ff284c
    BUG: 1081013
    Signed-off-by: Jeff Darcy <jdarcy>
    Reviewed-on: http://review.gluster.org/7360
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Niels de Vos <ndevos>
    Reviewed-by: Krishnan Parthasarathi <kparthas>
    Reviewed-by: Anand Avati <avati>

Comment 8 Anand Avati 2014-04-09 12:17:13 UTC
REVIEW: http://review.gluster.org/7361 (xlators/mgmt: don't leave zombies if required programs aren't installed) posted (#3) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 9 Anand Avati 2014-04-14 19:04:02 UTC
REVIEW: http://review.gluster.org/7361 (xlators/mgmt: don't leave zombies if required programs aren't installed) posted (#4) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 10 Anand Avati 2014-05-26 04:37:29 UTC
REVIEW: http://review.gluster.org/7361 (xlators/mgmt: don't leave zombies if required programs aren't installed) posted (#5) for review on master by Krishnan Parthasarathi (kparthas)

Comment 11 Anand Avati 2014-06-20 12:45:15 UTC
REVIEW: http://review.gluster.org/8134 (xlators/mgmt: don't allow glusterd fork bomb (cache the brick inode size)) posted (#1) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 12 Anand Avati 2014-06-20 14:56:00 UTC
REVIEW: http://review.gluster.org/8134 (xlators/mgmt: don't allow glusterd fork bomb (cache the brick inode size)) posted (#2) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 13 Anand Avati 2014-06-23 11:51:27 UTC
REVIEW: http://review.gluster.org/8134 (xlators/mgmt: don't allow glusterd fork bomb (cache the brick inode size)) posted (#3) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 14 Anand Avati 2014-06-23 14:20:04 UTC
REVIEW: http://review.gluster.org/8134 (xlators/mgmt: don't allow glusterd fork bomb (cache the brick inode size)) posted (#4) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 15 Anand Avati 2014-06-23 14:27:05 UTC
REVIEW: http://review.gluster.org/8134 (xlators/mgmt: don't allow glusterd fork bomb (cache the brick inode size)) posted (#5) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 16 Anand Avati 2014-06-25 05:58:43 UTC
COMMIT: http://review.gluster.org/8134 committed in master by Krishnan Parthasarathi (kparthas) 
------
commit f20d0ef8ad7d2f65a9234fc11101830873a9f6ab
Author: Kaleb S. KEITHLEY <kkeithle>
Date:   Fri Jun 20 08:40:47 2014 -0400

    xlators/mgmt: don't allow glusterd fork bomb (cache the brick inode size)
    
    Was don't leave zombies if required programs aren't installed
    
    Also, the existing if (strcmp (foo, bar) == 0) antipattern leaves me
    underwhelmed -- table driven is better;
    
    I like fully qualified paths to system tools too.
    
    File systems aren't going to change their inode size. Rather than
    fork-and-exec a tool repeatedly, hang on to the answer for subsequent
    use. Even if there are hundreds of volumes the size of a dict to keep
    this in memory is small.
    
    Change-Id: I704a8b1215446488b6e9e051a3e031af21b37adb
    BUG: 1081013
    Signed-off-by: Kaleb S. KEITHLEY <kkeithle>
    Reviewed-on: http://review.gluster.org/8134
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Krishnan Parthasarathi <kparthas>
    Tested-by: Krishnan Parthasarathi <kparthas>

Comment 17 Niels de Vos 2014-09-22 12:37:20 UTC
A beta release for GlusterFS 3.6.0 has been released. Please verify if the release solves this bug report for you. In case the glusterfs-3.6.0beta1 release does not have a resolution for this issue, leave a comment in this bug and move the status to ASSIGNED. If this release fixes the problem for you, leave a note and change the status to VERIFIED.

Packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update (possibly an "updates-testing" repository) infrastructure for your distribution.

[1] http://supercolony.gluster.org/pipermail/gluster-users/2014-September/018836.html
[2] http://supercolony.gluster.org/pipermail/gluster-users/

Comment 18 Niels de Vos 2014-11-11 08:29:17 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.6.1, please reopen this bug report.

glusterfs-3.6.1 has been announced [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://supercolony.gluster.org/pipermail/gluster-users/2014-November/019410.html
[2] http://supercolony.gluster.org/mailman/listinfo/gluster-users


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