From d3db942b217148767503284073f5d0dd4829d862 Mon Sep 17 00:00:00 2001 From: Aaron Mildenstein Date: Mon, 29 Feb 2016 10:00:34 -0700 Subject: [PATCH] Fix order of `--configtest` arg It should come before the `-f`, otherwise Logstash might bind a port and start running, rather than only perform a configtest, then exit. fixes 4737 Fixes #4738 --- pkg/logstash.sysv | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/logstash.sysv b/pkg/logstash.sysv index ce185988b..e255fe827 100755 --- a/pkg/logstash.sysv +++ b/pkg/logstash.sysv @@ -144,7 +144,7 @@ configtest() { HOME=${LS_HOME} export PATH HOME JAVA_OPTS LS_HEAP_SIZE LS_JAVA_OPTS LS_USE_GC_LOGGING - test_args="-f ${LS_CONF_DIR} --configtest ${LS_OPTS}" + test_args="--configtest -f ${LS_CONF_DIR} ${LS_OPTS}" $program ${test_args} [ $? -eq 0 ] && return 0 # Program not configured