diff options
| author | Raghavendra Talur <rtalur@redhat.com> | 2012-12-21 12:23:19 +0530 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2013-01-11 09:12:38 -0800 |
| commit | fd32c05f461efd48e46d879cdba85a2fe894a2e0 (patch) | |
| tree | cfdc0a3cb63339f664abc1710211391c988bb5a7 /extras/init.d | |
| parent | c5a70e5b1b646e5d5e6bec573c6729a7b149345d (diff) | |
extras: Changed the exit status in start function to 0.
Problem:
We were returning "1" when "glusterd start" was being called with service
already running. As per
http://refspecs.linuxbase.org/LSB_4.1.0/LSB-Core-generic/LSB-Core-generic/iniscrptact.html
we should consider it to be a success case and return 0.
Fix:
Just changed the return status from 1 to 0.
Change-Id: I8d6d2014f0c32ba53834503e721e2a71df7330a8
BUG: 845653
Signed-off-by: Raghavendra Talur <rtalur@redhat.com>
Reviewed-on: http://review.gluster.org/4348
Reviewed-by: Niels de Vos <ndevos@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Diffstat (limited to 'extras/init.d')
| -rwxr-xr-x | extras/init.d/glusterd-Redhat.in | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/extras/init.d/glusterd-Redhat.in b/extras/init.d/glusterd-Redhat.in index 5c92a4474a8..8b17eb18b8e 100755 --- a/extras/init.d/glusterd-Redhat.in +++ b/extras/init.d/glusterd-Redhat.in @@ -35,7 +35,7 @@ start() status=$? if [ $status -eq 0 ]; then echo "glusterd service is already running with pid $PID" - exit 1 + exit 0 else echo -n $"Starting $BASE:" daemon $GLUSTERD |