Skip to content

Commit

Permalink
Fix configtest option in init scripts
Browse files Browse the repository at this point in the history
No need to export all the JAVA* related environment variables
for configtest. It needs a minimal set of variables to execute
bin/logstash --config-test. This was indirectly causing issues
when used with JMX options in LS_JAVA_OPTS. JMX needs a remote port defined
and configtest was trying to connect to the port twice which caused
restart (#4319) to fail

Fixes #4319

Fixes #4759
  • Loading branch information
suyograo committed Mar 4, 2016
1 parent 29038fb commit 96a5457
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/logstash.sysv
Expand Up @@ -142,7 +142,7 @@ configtest() {
fi

HOME=${LS_HOME}
export PATH HOME JAVA_OPTS LS_HEAP_SIZE LS_JAVA_OPTS LS_USE_GC_LOGGING
export PATH HOME

test_args="--configtest -f ${LS_CONF_DIR} ${LS_OPTS}"
$program ${test_args}
Expand Down

0 comments on commit 96a5457

Please sign in to comment.