summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKotresh HR <khiremat@redhat.com>2015-03-23 15:07:47 +0530
committerVijay Bellur <vbellur@redhat.com>2015-04-06 10:44:35 -0700
commit85865daa1b7dd11badf9f5192e050e1998c76f8a (patch)
tree0526d2d1e5910ee3d50c7ea74550b4735dc47b1c
parent97cfe210c73bcdab653a0da884627009f5c5808b (diff)
extras: Fix stop-all-gluster-processes.sh script
"test -n" command takes single string as argument. The command was failing with "Too many arguments" when multiple pids are got. Change-Id: Icc409082f492c72522168d5e203684f00f52cf1b BUG: 1204641 Signed-off-by: Kotresh HR <khiremat@redhat.com> Reviewed-on: http://review.gluster.org/9970 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Aravinda VK <avishwan@redhat.com> Reviewed-by: Vijay Bellur <vbellur@redhat.com>
-rwxr-xr-xextras/stop-all-gluster-processes.sh4
1 files changed, 2 insertions, 2 deletions
diff --git a/extras/stop-all-gluster-processes.sh b/extras/stop-all-gluster-processes.sh
index 6c81a301c72..087afa4cf22 100755
--- a/extras/stop-all-gluster-processes.sh
+++ b/extras/stop-all-gluster-processes.sh
@@ -13,7 +13,7 @@ function main()
# processes are not having a pid file, so get it through 'ps' and
# handle these processes
gsyncpid=`ps aux | grep gluster | grep gsync | awk '{print $2}'`;
- test -n $gsyncpid && kill -TERM $gsyncpid;
+ test -n "$gsyncpid" && kill -TERM $gsyncpid;
sleep 5;
@@ -27,7 +27,7 @@ function main()
# handle 'KILL' of geo-replication
gsyncpid=`ps aux | grep gluster | grep gsync | awk '{print $2}'`;
- test -n $gsyncpid && kill -KILL $gsyncpid;
+ test -n "$gsyncpid" && kill -KILL $gsyncpid;
}
main "$@";