Bug 1277562

Summary: stop-all-gluster-processes.sh doesn't return correct return status
Product: [Red Hat Storage] Red Hat Gluster Storage Reporter: Ramesh N <rnachimu>
Component: glusterfsAssignee: Ramesh N <rnachimu>
Status: CLOSED CURRENTRELEASE QA Contact: Bhaskarakiran <byarlaga>
Severity: unspecified Docs Contact:
Priority: medium    
Version: rhgs-3.1CC: asrivast, byarlaga, chrisw, csaba, mzywusko, nlevinki, rcyriac, rhs-bugs, sabose, sankarshan, vbellur
Target Milestone: ---Keywords: ZStream
Target Release: RHGS 3.1.2   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: glusterfs-3.7.5-7 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1277533 Environment:
Last Closed: 2016-07-07 12:54:26 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: 1277533, 1279776    
Bug Blocks: 1189285, 1260783, 1291173    

Description Ramesh N 2015-11-03 14:44:27 UTC 
+++ This bug was initially created as a clone of Bug #1277533 +++

Description of problem:

stop-all-gluster-processes.sh doesn't return correct return status after successful completion.

How reproducible:

Always.

Steps to Reproduce:
1. run "stop-all-gluster-processes.sh" to kill all gluster process
2. # echo $?


Actual results:

 1

Expected results:

0

--- Additional comment from Vijay Bellur on 2015-11-03 09:12:20 EST ---

REVIEW: http://review.gluster.org/11739 (extras: Exit with SUCCESS if no processes to stop) posted (#7) for review on master by Ramesh N (rnachimu)

--- Additional comment from Vijay Bellur on 2015-11-03 09:38:44 EST ---

REVIEW: http://review.gluster.org/11739 (extras: Exit with SUCCESS if no processes to stop) posted (#8) for review on master by Ramesh N (rnachimu)
Comment 3 Bhaskarakiran 2015-11-25 11:39:46 UTC 
[root@ninja ~]# /usr/share/glusterfs/scripts/stop-all-gluster-processes.sh ; echo $?
sending SIGTERM to process 8948
sending SIGTERM to process 8904
sending SIGTERM to process 21469
sending SIGTERM to process 28853
0
[root@ninja ~]# 
[root@ninja ~]# gluster --version
glusterfs 3.7.5 built on Nov 24 2015 02:22:29
Repository revision: git://git.gluster.com/glusterfs.git
Copyright (c) 2006-2011 Gluster Inc. <http://www.gluster.com>
GlusterFS comes with ABSOLUTELY NO WARRANTY.
You may redistribute copies of GlusterFS under the terms of the GNU General Public License.
[root@ninja ~]#