Skip to content

Commit

Permalink
fix(docker): Set proper variable expansion for JMX_OPTS and JAVA_OPTS…
Browse files Browse the repository at this point in the history
… in MAE docker (datahub-project#6378)

image
  • Loading branch information
skrydal authored and cccs-Dustin committed Feb 1, 2023
1 parent 2142ea5 commit 7371d9e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion docker/datahub-mae-consumer/start.sh
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ if [[ ${GRAPH_SERVICE_IMPL:-} != elasticsearch ]] && [[ ${SKIP_NEO4J_CHECK:-fals
dockerize_args+=("-wait" "$NEO4J_HOST")
fi

JAVA_TOOL_OPTIONS="${JDK_JAVA_OPTIONS:-}${JAVA_OPTS:+ JAVA_OPTS}${JMX_OPTS:+ JMX_OPTS}"
JAVA_TOOL_OPTIONS="${JDK_JAVA_OPTIONS:-}${JAVA_OPTS:+ $JAVA_OPTS}${JMX_OPTS:+ $JMX_OPTS}"
if [[ ${ENABLE_OTEL:-false} == true ]]; then
JAVA_TOOL_OPTIONS="$JAVA_TOOL_OPTIONS -javaagent:opentelemetry-javaagent-all.jar"
fi
Expand Down

0 comments on commit 7371d9e

Please sign in to comment.