Bug 1335726 - stop all gluster processes should also also include glusterfs mount process
Summary: stop all gluster processes should also also include glusterfs mount process
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: scripts
Version: 3.8.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
Depends On: 1334620 1334750
Blocks: 1330975
TreeView+ depends on / blocked
 
Reported: 2016-05-13 06:17 UTC by Prasanna Kumar Kalever
Modified: 2016-06-16 14:06 UTC (History)
1 user (show)

Fixed In Version: glusterfs-3.8rc2
Clone Of: 1334750
Environment:
Last Closed: 2016-06-16 14:06:29 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Prasanna Kumar Kalever 2016-05-13 06:17:02 UTC
Description of problem:
for cases such as umount return success and glusterfs process still not quit, stop-all-gluster-processes.sh should handle this to do a force action

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

Actual results:
after executing stop-all-gluster-processes.sh we still see mount process

Expected results:
clean mount process as well

Comment 1 Vijay Bellur 2016-05-13 06:18:04 UTC
REVIEW: http://review.gluster.org/14321 (extras: stop all include glusterfs process as well) posted (#1) for review on release-3.8 by Prasanna Kumar Kalever (pkalever)

Comment 2 Vijay Bellur 2016-05-13 06:44:45 UTC
REVIEW: http://review.gluster.org/14321 (extras: stop all include glusterfs process as well) posted (#2) for review on release-3.8 by Prasanna Kumar Kalever (pkalever)

Comment 3 Vijay Bellur 2016-05-14 07:30:09 UTC
REVIEW: http://review.gluster.org/14321 (extras: stop all include glusterfs process as well) posted (#3) for review on release-3.8 by Prasanna Kumar Kalever (pkalever)

Comment 4 Vijay Bellur 2016-05-15 07:07:37 UTC
COMMIT: http://review.gluster.org/14321 committed in release-3.8 by Niels de Vos (ndevos) 
------
commit 1ab8c44c56a13dc3fbc948bdfc1a3dfe80d66da9
Author: Prasanna Kumar Kalever <prasanna.kalever>
Date:   Tue May 10 12:59:56 2016 +0530

    extras: stop all include glusterfs process as well
    
    currently, extras/stop-all-gluster-processes.sh script handles
    brick processes, node services and geo-rep's gsync process.
    
    from now this script also handles mount processes as well,
    as part of this patch I have reorganized this script
    
    Backport of:
    > Change-Id: Id62d6fda6dd331bde722ce3d99ec3f09fed55cb0
    > BUG: 1334620
    > Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever>
    > Reviewed-on: http://review.gluster.org/14277
    > Tested-by: Prasanna Kumar Kalever <pkalever>
    > Smoke: Gluster Build System <jenkins.com>
    > NetBSD-regression: NetBSD Build System <jenkins.org>
    > Reviewed-by: Niels de Vos <ndevos>
    > CentOS-regression: Gluster Build System <jenkins.com>
    
    Change-Id: Id62d6fda6dd331bde722ce3d99ec3f09fed55cb0
    BUG: 1335726
    Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever>
    Reviewed-on: http://review.gluster.org/14321
    Tested-by: Prasanna Kumar Kalever <pkalever>
    Smoke: Gluster Build System <jenkins.com>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.com>
    Reviewed-by: Kaleb KEITHLEY <kkeithle>
    Reviewed-by: Niels de Vos <ndevos>

Comment 5 Niels de Vos 2016-06-16 14:06:29 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.8.0, please open a new bug report.

glusterfs-3.8.0 has been announced on the Gluster mailinglists [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://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user


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